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.

Sign up for GitHub

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

Jump to bottom

[18.0][MIG] base_user_role: Migration to version 18.0 #313

Merged
merged 142 commits into from
Nov 27, 2024
Merged

[18.0][MIG] base_user_role: Migration to version 18.0 #313

merged 142 commits into from
Nov 27, 2024

Conversation

BertVGroenendael
Copy link

/ocabot migration base_user_role

sebalix and others added 30 commits October 21, 2024 08:09
@sebalix @BertVGroenendael
8.0 - New module 'base_user_role' to manage user roles efficiently (#…
7f54d56 
…608)

* [ADD] New module 'base_user_role'

* [FIX] base_user_role - Review

* [FIX] base_user_role - Review s/is_active/is_enabled/

* [FIX] base_user_role - Review s/is_active/is_enabled/

* [IMP] base_user_role - Translations updated (template + FR)

* [FIX] base_user_role - Lint
@sbidoul @BertVGroenendael
base_user_role: display roles as first tab in user form
d4439e9
@sebalix @BertVGroenendael
base_user_role - Set a default category to associated groups
3df52a1
@adrienpeiffer @BertVGroenendael
[MIG] base_user_role: Port to 10.0
575014d
@alexis-via @BertVGroenendael
[FIX] Crash in base_user_role when role_id is not set on res.users.ro…
f983522 
…le.line
@benwillig @BertVGroenendael
[10.0][FIX] User group not updating when removing a user from the rol…
29362f1 
…e view
@BertVGroenendael
[MIG] base_user_role: Migration to 11.0
185d344
@sebalix @BertVGroenendael
[FIX+IMP] base_user_role: Several things:
1c2d2c6 
* FIX: Update users' groups when a role is deleted
* IMP: add some test cases to check user's groups when its last role line is removed
* FIX: ensure that the user's groups are updated when its last role line is removed
@BertVGroenendael
[MIG] Update test cases for testing (unlink role and role line) to ne…
3142b9e 
…w api
@benwillig @BertVGroenendael
[IMP] Use default user to define default roles to create for a new user
2da61ca
@oca-travis @BertVGroenendael
[UPD] Update base_user_role.pot
fe85d9e
@hhgabelgaard @BertVGroenendael
Translated using Weblate (Danish)
95c781a 
Currently translated at 100,0% (37 of 37 strings)

Translation: server-backend-11.0/server-backend-11.0-base_user_role
Translate-URL: https://translation.odoo-community.org/projects/server-backend-11-0/server-backend-11-0-base_user_role/da/
@BertVGroenendael
[MIG] base_user_role: Migration to 12.0
da0197b
@OCA-git-bot @BertVGroenendael
[UPD] README.rst
bcad476
@oca-travis @BertVGroenendael
[UPD] Update base_user_role.pot
29d22b6
@oca-transbot @BertVGroenendael
Update translation files
14181e7 
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
@OCA-git-bot @BertVGroenendael
[UPD] README.rst
1ef3ecf
@HviorForgeFlow @BertVGroenendael
[12.0][FIX] readme
6319b5e
@OCA-git-bot @BertVGroenendael
[UPD] README.rst
6170a1c
@ernestotejeda @BertVGroenendael
[MIG] base_import_match: Migration to 12.0
134c1e7
@JordiBForgeFlow @BertVGroenendael
[base_user_role] improve performance by only adding/removing from a user
48b77b2 
the groups that have changed with respecto the role.
@OCA-git-bot @BertVGroenendael
[UPD] README.rst
495553d
@artux404 @BertVGroenendael
[10.0][UPD] base_user_role: Disabled Administrator user selection in …
c3c8825 
…notebook users line (#1170)
@elicoidal @BertVGroenendael
Switch AGPL to LGPL license
fc34701
@BertVGroenendael
Revert "[FIX] Crash in base_user_role when role_id is not set on res.…
ce61293 
…users.role.line"

This reverts commit b537941d5a2d1fcb3fbacc602124c1b3f6e1d495.
@BertVGroenendael
Make user_id and role_id required in base_user_role_line
8cdda85
@BertVGroenendael
[IMP] make base_user_role_line company dependent
9bf77f5
@BertVGroenendael
FIX check_company
155a70e
@sebalix @BertVGroenendael
[MIG] base_user_role: Migration to 13.0
10bcb1d
@sebalix @BertVGroenendael
[IMP] base_user_role: black, isort
a67a8aa
weblate and others added 7 commits October 21, 2024 08:09
@weblate @BertVGroenendael
Update translation files
8db4128 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-backend-17.0/server-backend-17.0-base_user_role
Translate-URL: https://translation.odoo-community.org/projects/server-backend-17-0/server-backend-17-0-base_user_role/
@mymage @BertVGroenendael
Translated using Weblate (Italian)
e2ee5d8 
Currently translated at 100.0% (65 of 65 strings)

Translation: server-backend-17.0/server-backend-17.0-base_user_role
Translate-URL: https://translation.odoo-community.org/projects/server-backend-17-0/server-backend-17-0-base_user_role/it/
@BertVGroenendael
[FIX] base_user_role: prevent crash when using 'res.users' (x2m) fiel…
efb7791 
…ds in forms
@OCA-git-bot @BertVGroenendael
[BOT] post-merge updates
0fceb49
@BertVGroenendael
[IMP] base_user_role: pre-commit stuff
7acd964
@BertVGroenendael
[IMP] base_user_role: pre-commit stuff
9cdb798
@BertVGroenendael
[18.0][MIG] base_user_role
8f26ebb
@BertVGroenendael
Copy link
Author

/ocabot migration base_user_role

@OCA-git-bot
Copy link
Contributor

Sorry @BertVGroenendael you are not allowed to mark the addon tobe migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

This was referenced Oct 21, 2024
Open
Closed
BertVGroenendael added a commit to DynAppsNV/server-backend that referenced this pull request Oct 24, 2024
@BertVGroenendael
[ADD] test-requirements.txt on PR OCA#313
b43be47
BertVGroenendael added a commit to DynAppsNV/server-backend that referenced this pull request Oct 24, 2024
@BertVGroenendael
[ADD] test-requirements.txt on PR OCA#313
697aa95
@StefanRijnhart
Copy link
Member

/ocabot migration base_user_role

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Nov 8, 2024
SiesslPhillip pushed a commit to grueneerde/OCA-server-backend that referenced this pull request Nov 20, 2024
@bt-admin
Merge pull request OCA#313 from OCA/16.0
41403d9 
Syncing from upstream OCA/server-backend (16.0)
@mylbco
Copy link

mylbco commented Nov 27, 2024

Functional test ok !

StefanRijnhart
StefanRijnhart approved these changes Nov 27, 2024
View reviewed changes
Copy link
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

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

Thanks!

mylbco
mylbco approved these changes Nov 27, 2024
View reviewed changes
Copy link

@mylbco mylbco left a comment

Choose a reason for hiding this comment

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

Functional test

@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). 🤖

@StefanRijnhart
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-313-by-StefanRijnhart-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 2de96b5 into OCA:18.0 Nov 27, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@ristecona ristecona ristecona approved these changes

@mylbco mylbco mylbco approved these changes

@StefanRijnhart StefanRijnhart StefanRijnhart approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
18.0
Development

Successfully merging this pull request may close these issues.