[19.0][MIG] base_user_role: Migration to 19.0
Scope
- base_user_role only
Summary
- Migrate to Odoo 19.0 following OCA guide.
- API updates:
group_idsrename,SUPERUSER_IDimport fromodoo.api. - Switch to
models.Constraintandmodels.IndexAPIs. - Switch to
res.groupsprivilege_id(dropcategory_id). - Replace
trans_implied_idsusages withall_implied_idswhere effective groups are expected. - Adjust search/tree view xpaths and
<search><group>schema for 19.0. - Refactor default roles due to removal of default user in Odoo.
Pre-commit
- Verified locally; auto-fixes applied and committed.
Tests
Command:
./odoo/odoo-bin -c odoo.conf -d bur19_tests_bur -i base_user_role --test-enable --test-tags "/base_user_role" --without-demo=all --log-level=info --stop-after-init
Result: green locally.
Base migration PR for the role stack. Two downstream PRs depend on this PR via a temporary test requirement, following OCA guidance:
https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference(s)-to-another-pull-request(s)
Dependent PRs
- #383 base_user_role_company
- #384 base_user_role_history
Notes
- The dependents include a commit titled "[DON'T MERGE] test-requirements.txt"; we will drop that commit once this base PR is merged.
- Local tests are green. Command used:
./odoo/odoo-bin -c odoo.conf -d bur19_tests_bur -i base_user_role --test-enable --test-tags "/base_user_role" --without-demo=all --log-level=info --stop-after-init
Maintainers ping
- @sebalix @jcdrubay @novawish — thanks in advance for an initial look. This PR is kept as draft; happy to mark ready once you prefer.
/ocabot migration base_user_role
@tishmen Is this ready ? Some of commits can be squashed in migration one. If there are improvements, they can stay but making module name appear in commit message. Thanks
@rousseldenis From my side this is ready for another look; The module is installing and checks/tests are passing. I’ll squash the migration-only changes into a single “[19.0] [MIG] …” commit and ping you once done.
@FrancoMaxime @rousseldenis The commits have been squashed as suggested, please check the last commit for the full diff. Thank you!
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). 🤖
/ocabot merge nobump
Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 19.0-ocabot-merge-pr-382-by-StefanRijnhart-bump-nobump, awaiting test results.
Congratulations, your PR was merged at 9b05edd17fd1bf67a3ef0903c19c64b6d8adcf6b. Thanks a lot for contributing to OCA. ❤️