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

[18.0][MIG] hr_employee_second_lastname: Migration to 18.0 #1398

Merged
merged 8 commits into from
Nov 19, 2024

Conversation

edy1192
Copy link
Contributor

@edy1192 edy1192 commented Oct 31, 2024

No description provided.

@edy1192 edy1192 marked this pull request as draft October 31, 2024 23:10
@edy1192 edy1192 force-pushed the 18.0-mig-hr-employee-second-lastname branch from 047a002 to 7f20b88 Compare October 31, 2024 23:17
andreagidaltig and others added 8 commits October 31, 2024 23:33
Module is renamed from hr_employee_lastnames → hr_employee_second_lastname
in order to be consistent with partner_second_lastname module.
- Fix some new lints.
- Update incoming parameters of post_init_hook method since `cr, registry`
  were replaced by `env` in [1].
- Deprecate the use of _onchange_spec method and use the method new method
  _get_fields_spec to return the fields specification from a view
  description since it was introduced in [2].

[1] odoo/odoo@b4a7996
[2] odoo/odoo@f5e6494
@edy1192 edy1192 force-pushed the 18.0-mig-hr-employee-second-lastname branch from 7f20b88 to 833abf0 Compare October 31, 2024 23:33
@edy1192 edy1192 marked this pull request as ready for review November 1, 2024 19:05
@edy1192
Copy link
Contributor Author

edy1192 commented Nov 1, 2024

Hi @luisg123v @luistorresm @andreagidaltig could you check this, please?

Regards.

Copy link
Contributor

@luistorresm luistorresm left a comment

Choose a reason for hiding this comment

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

Thanks @edy1192

@andreagidaltig
Copy link
Contributor

Hi @luisg123v @luistorresm @andreagidaltig could you check this, please?

Regards.

LGTM 👍

@edy1192
Copy link
Contributor Author

edy1192 commented Nov 6, 2024

Hi @luisg123v could you check this, please?

Copy link
Contributor

@luisg123v luisg123v left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@moylop260 could you review/merge, please?

Remember to run the command to mark this as a migration PR.

@moylop260
Copy link

/ocabot migration hr_employee_second_lastname

@moylop260
Copy link

/ocabot merge nobump

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

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-1398-by-moylop260-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request Nov 19, 2024
13 tasks
@OCA-git-bot OCA-git-bot merged commit c3f0051 into OCA:18.0 Nov 19, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e316bcc. 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.

7 participants