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

[14.0][FIX] base_user_role_company: wrong menus on re-login #149

Merged
merged 1 commit into from
Mar 3, 2022

Conversation

dreispt
Copy link
Member

@dreispt dreispt commented Mar 2, 2022

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.

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.
@dreispt dreispt changed the title [FIX] base_user_role_company: wrong menus on re-login [14.0][FIX] base_user_role_company: wrong menus on re-login Mar 2, 2022
Copy link

@patrickrwilson patrickrwilson left a comment

Choose a reason for hiding this comment

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

Tested and works great.

@dreispt
Copy link
Member Author

dreispt commented Mar 2, 2022

(To be clear, this needs #147 to work properly, of course.)

@dreispt
Copy link
Member Author

dreispt commented Mar 3, 2022

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-149-by-dreispt-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 33a363c into OCA:14.0 Mar 3, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a5b6aed. Thanks a lot for contributing to OCA. ❤️

SiesslPhillip pushed a commit to grueneerde/OCA-server-backend that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-backend (15.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants