-
-
Notifications
You must be signed in to change notification settings - Fork 673
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[17.0][MIG] hr_employee_relative: Migration to 17.0 #1382
Conversation
Currently translated at 100.0% (32 of 32 strings) Translation: hr-14.0/hr-14.0-hr_employee_relative Translate-URL: https://translation.odoo-community.org/projects/hr-14-0/hr-14-0-hr_employee_relative/pt_BR/
Currently translated at 93.7% (30 of 32 strings) Translation: hr-14.0/hr-14.0-hr_employee_relative Translate-URL: https://translation.odoo-community.org/projects/hr-14-0/hr-14-0-hr_employee_relative/de/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: hr-16.0/hr-16.0-hr_employee_relative Translate-URL: https://translation.odoo-community.org/projects/hr-16-0/hr-16-0-hr_employee_relative/
Module migrated to version 17.0 cc https://github.com/APSL 158255 @miquelalzanillas @lbarry-apsl @mpascuall @peluko00 @javierobcn @BernatObrador please review Issue #1291 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, code review and tested in runboat
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, code review and tested in runboat
/ocabot migration hr_employee_relative |
This PR has the |
Hey @pedrobaeza by any chance, do you know who is the maintainer or can you please help us out with this merge? Thanks in advance. Regards, |
@Saran440 may help. I see controversial to strip the years float field to integer. |
Thanks you! @pedrobaeza . @Saran440 your feedback is really important and welcomed here. So this PR can be merge. Thanks. |
@andrp92 I think we should add separate fields to store What do you think? However, I tested this module and it worked. 👍 |
@Saran440 It sounds reasonable. However, please notice that it was already managed in a single field back in:
But yes, I think that can be improved. What do you think @ppyczko? |
1b93777
to
b5eb869
Compare
@ppyczko Could you split commit
|
Currently translated at 15.6% (5 of 32 strings) Translation: hr-16.0/hr-16.0-hr_employee_relative Translate-URL: https://translation.odoo-community.org/projects/hr-16-0/hr-16-0-hr_employee_relative/it/
Currently translated at 100.0% (32 of 32 strings) Translation: hr-16.0/hr-16.0-hr_employee_relative Translate-URL: https://translation.odoo-community.org/projects/hr-16-0/hr-16-0-hr_employee_relative/es/
Currently translated at 100.0% (32 of 32 strings) Translation: hr-16.0/hr-16.0-hr_employee_relative Translate-URL: https://translation.odoo-community.org/projects/hr-16-0/hr-16-0-hr_employee_relative/fr/
Currently translated at 100.0% (32 of 32 strings) Translation: hr-16.0/hr-16.0-hr_employee_relative Translate-URL: https://translation.odoo-community.org/projects/hr-16-0/hr-16-0-hr_employee_relative/pt_BR/
b5eb869
to
df206b1
Compare
@Saran440 done, thanks! |
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 44c086b. Thanks a lot for contributing to OCA. ❤️ |
No description provided.