[18.0][MIG] default_multi_user: Migration to 18.0 #1130
Conversation
is incorrect to relay in the company_id field in the res_users table, we should use environment company instead.
Currently translated at 100.0% (8 of 8 strings) Translation: server-ux-14.0/server-ux-14.0-default_multi_user Translate-URL: https://translation.odoo-community.org/projects/server-ux-14-0/server-ux-14-0-default_multi_user/es/
Currently translated at 100.0% (8 of 8 strings) Translation: server-ux-14.0/server-ux-14.0-default_multi_user Translate-URL: https://translation.odoo-community.org/projects/server-ux-14-0/server-ux-14-0-default_multi_user/pt/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-ux-15.0/server-ux-15.0-default_multi_user Translate-URL: https://translation.odoo-community.org/projects/server-ux-15-0/server-ux-15-0-default_multi_user/
This was fixed upstream but the signature of the method never got updated here in the corresponding hook.
There was a problem hiding this comment.
When migrating modules with hooks that modify original methods, you need to make sure that nothing is changes apart from what's strictly the purpose (in this case highlighted with comments # START/END OF CHANGES. Apart from that there is no intention of modifying the orignal method. In v18 the method changed slightly, and that change needs to be also included here: https://github.com/odoo/odoo/blob/18.0/odoo/addons/base/models/ir_default.py#L159
ArnauCForgeFlow
force-pushed
the
18.0-mig-default_multi_user
branch
from
f0c8d95 to
9229b35
Compare
|
@LoisRForgeFlow Done! |
|
/ocabot migration default_multi_user |
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at fa69717. Thanks a lot for contributing to OCA. ❤️ |
Standard Migration to version 18.0