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

Custom menu permissions in CP only look at primary role #3270

Open
robinsowell opened this issue Apr 13, 2023 · 2 comments · May be fixed by #3719
Open

Custom menu permissions in CP only look at primary role #3270

robinsowell opened this issue Apr 13, 2023 · 2 comments · May be fixed by #3719
Labels

Comments

@robinsowell
Copy link
Contributor

I made a custom menu and set permissions so only superadmins have access.

I assigned a member to primary role of 'Editor' and then in role assignment, they're assigned to both Editor and Superadmin roles.

If I login as this member in the CP, I don't see my custom menu, even though they have superadmin as a secondary role.

Replicated EE 7.2.14

@robinsowell robinsowell added the Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. label Apr 13, 2023
@intoeetive intoeetive removed the Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. label Apr 14, 2023
@intoeetive
Copy link
Contributor

@robinsowell IDK if we should call it a bug, with the way how custom menus work now

If you create two menus, and try to assign them to same role, you'll see no error, but the assigment for the "older" one will be erased. And only one menu will be displayed.

So it looks like the custom menus are built with the idea that only one menu can be displayed at a time. And that menu is the one that's for primary role.

We might want to rework this - like, display each menu separately in the sidebar, with the menu name instead of "custom". But we probably will want to discuss this with @TomJaeger

@intoeetive intoeetive added the UX label Apr 14, 2023
@intoeetive
Copy link
Contributor

Related discussion: #3572

@intoeetive intoeetive linked a pull request Dec 7, 2023 that will close this issue
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 a pull request may close this issue.

2 participants