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] base_user_role_company: Migration to 18.0 #315

Closed

Conversation

BertVGroenendael
Copy link

No description provided.

Chandresh-OSI and others added 30 commits October 22, 2024 15:57
Currently translated at 100.0% (11 of 11 strings)

Translation: server-backend-14.0/server-backend-14.0-base_user_role_company
Translate-URL: https://translation.odoo-community.org/projects/server-backend-14-0/server-backend-14-0-base_user_role_company/it/
Issue found on logout / relogin.
The user groups were applied correctly, but the main menu showed apps
the user did not have access to.

This was related to the menu caching mechanisn, that was disabled here.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-backend-14.0/server-backend-14.0-base_user_role_company
Translate-URL: https://translation.odoo-community.org/projects/server-backend-14-0/server-backend-14-0-base_user_role_company/
Currently translated at 90.9% (10 of 11 strings)

Translation: server-backend-14.0/server-backend-14.0-base_user_role_company
Translate-URL: https://translation.odoo-community.org/projects/server-backend-14-0/server-backend-14-0-base_user_role_company/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-backend-16.0/server-backend-16.0-base_user_role_company
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_user_role_company/
Currently translated at 100.0% (8 of 8 strings)

Translation: server-backend-16.0/server-backend-16.0-base_user_role_company
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_user_role_company/es/
Currently translated at 100.0% (8 of 8 strings)

Translation: server-backend-16.0/server-backend-16.0-base_user_role_company
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_user_role_company/pt/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-backend-16.0/server-backend-16.0-base_user_role_company
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_user_role_company/
@BertVGroenendael BertVGroenendael force-pushed the 18.0-mig-base_user_role_company branch from b43be47 to 697aa95 Compare October 24, 2024 08:36
@ristecona
Copy link

do we have movement on this ? Thanks for the update

@StefanRijnhart
Copy link
Member

/ocabot migration base_user_role_company

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Nov 8, 2024
@StefanRijnhart
Copy link
Member

@ristecona You can help! First step would be to test or review this one: #313, after which we can consider this one.

@ristecona
Copy link

@StefanRijnhart i want to review but i dont have permissions to do so :(

@StefanRijnhart
Copy link
Member

@ristecona I did not know special permissions were required. Do you not see the green "Review Changes" button on the "Files Changed" tab?
image

@ristecona
Copy link

@StefanRijnhart nope i dont have that at all

@StefanRijnhart
Copy link
Member

@ristecona It's a bit of a mystery. I just tested with a brand new dummy Github account and I have access to the button so I don't understand why you should not have access to it.

@ristecona
Copy link

@StefanRijnhart done!

SiesslPhillip pushed a commit to grueneerde/OCA-server-backend that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-backend (17.0)
@StefanRijnhart
Copy link
Member

@BertVGroenendael Can you remove the test dependency now that #313 is merged?

Copy link

@vvrossem vvrossem left a comment

Choose a reason for hiding this comment

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

pre-approving

Copy link

@vvrossem vvrossem Jan 16, 2025

Choose a reason for hiding this comment

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

this file (and its related commit) can be removed now that #313 is merged

@StefanRijnhart
Copy link
Member

Closing in favor of #328. Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.