-
-
Notifications
You must be signed in to change notification settings - Fork 672
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
[18.0][MIG] hr_employee_second_lastname: Migration to 18.0 #1398
Conversation
047a002
to
7f20b88
Compare
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
7f20b88
to
833abf0
Compare
Hi @luisg123v @luistorresm @andreagidaltig could you check this, please? Regards. |
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.
Thanks @edy1192
LGTM 👍 |
Hi @luisg123v could you check this, please? |
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 👍
@moylop260 could you review/merge, please?
Remember to run the command to mark this as a migration PR.
/ocabot migration hr_employee_second_lastname |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at e316bcc. Thanks a lot for contributing to OCA. ❤️ |
No description provided.