-
-
Notifications
You must be signed in to change notification settings - Fork 304
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
[16.0][IMP]base_rest: show first-level menu to admins only #477
base: 16.0
Are you sure you want to change the base?
Conversation
9e44bfe
to
321bf80
Compare
@lmignon I'm trying to investigate the broken tests but they fail even on the |
@PicchiSeba I don't understand the problem. On my laptop all the tests succeed. I need to analyze this problem in depth. |
something may have changed in Odoo and as I don't have the latest version on my machine, I can't see it. Unfortunately, I don't have the time right now to analyze the problem further. |
Understandable, thanks for the answer regardless |
321bf80
to
1597126
Compare
1597126
to
7c37639
Compare
As stated in #470, tests were broken because of a change in upstream odoo |
I was thinking that maybe is it better to show it with developer mode enabled + admin settings? Just a guess, but this PR still definitely improves the situation right now (shown to all, even non-technical users or normal employees). |
fw-port of #425 to
16.0