Skip to content
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

Merged
merged 24 commits into from
Sep 6, 2024

Conversation

ppyczko
Copy link
Contributor

@ppyczko ppyczko commented Jul 17, 2024

No description provided.

alexey-pelykh and others added 16 commits July 17, 2024 07:50
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/
@ppyczko ppyczko mentioned this pull request Jul 17, 2024
22 tasks
@ppyczko
Copy link
Contributor Author

ppyczko commented Jul 17, 2024

Module migrated to version 17.0

cc https://github.com/APSL 158255

@miquelalzanillas @lbarry-apsl @mpascuall @peluko00 @javierobcn @BernatObrador please review

Issue #1291

Copy link
Contributor

@peluko00 peluko00 left a 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

Copy link

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@BernatObrador BernatObrador left a 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

@Saran440
Copy link
Member

/ocabot migration hr_employee_relative

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Jul 19, 2024
@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@andres-pcg
Copy link

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,

@pedrobaeza
Copy link
Member

@Saran440 may help. I see controversial to strip the years float field to integer.

@andres-pcg
Copy link

Thanks you! @pedrobaeza .

@Saran440 your feedback is really important and welcomed here. So this PR can be merge. Thanks.

@Saran440
Copy link
Member

Saran440 commented Sep 4, 2024

@andrp92 I think we should add separate fields to store age_year, age_month, and age_day to capture all details.
This way, we can accommodate any projects or countries that might require age in terms of months or days. (allow invisible it if not need)

What do you think?

However, I tested this module and it worked. 👍

@andres-pcg
Copy link

andres-pcg commented Sep 4, 2024

@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?

@ppyczko ppyczko force-pushed the 17.0-mig-hr_employee_relative branch 4 times, most recently from 1b93777 to b5eb869 Compare September 5, 2024 07:14
@ppyczko
Copy link
Contributor Author

ppyczko commented Sep 5, 2024

@Saran440 @andrp92 thank you for your feedback! I added the requested changes, let me know if there's anything else!

@Saran440
Copy link
Member

Saran440 commented Sep 6, 2024

@ppyczko Could you split commit

  1. [MIG] --> for standard migration
  2. [ENH] --> for enhance field float to integer. we will can backport this commit.

mymage and others added 8 commits September 6, 2024 07:16
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/
@ppyczko ppyczko force-pushed the 17.0-mig-hr_employee_relative branch from b5eb869 to df206b1 Compare September 6, 2024 05:30
@ppyczko
Copy link
Contributor Author

ppyczko commented Sep 6, 2024

@Saran440 done, thanks!

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-1382-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit ed8d16e into OCA:17.0 Sep 6, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 44c086b. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.