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

Channels' Publish Layouts don't respect non-Primary member roles #2298

Closed
mcfarlandonline opened this issue Aug 18, 2022 · 3 comments
Closed

Comments

@mcfarlandonline
Copy link
Contributor

EE 6.3.5
PHP 7.3.33

I have a website where all the members need to have the same Primary role so they can access a front-facing intranet. However, some of those members also need Site Admin clearance, so they're assigned those roles, too. (But only the minimum access role as their "Primary" role).

I just noticed today, that the channel Publish Layouts don't honor the secondary member roles when they're selected. So my Site Admins aren't actually benefiting from a custom/clean interface.

@mcfarlandonline mcfarlandonline changed the title Channel Publish Layouts don't respect non-Primary member roles Channels' Publish Layouts don't respect non-Primary member roles Aug 18, 2022
@TomJaeger
Copy link
Contributor

Just to make sure we follow, users with the Super Admin role are getting the Super Admin publish layout, even if that's NOT their primary role?

@mcfarlandonline
Copy link
Contributor Author

No, it seems even for the Super Admins, that if they're Primary Role is something else, they don't get the Super Admin publish layout.

So, for example, this is my Super Admin's role setting...

(Not pictured: in addition to Super Admins, the Member Portal role is reiterated/checked in the "secondary" Roles checkbox menu).

Screen Shot 2022-08-18 at 1 39 58 PM

And this is the channel's Publish Layout setting...

Screen Shot 2022-08-18 at 1 40 17 PM

But my Super Admin doesn't reap the benefits of this layout.

@intoeetive
Copy link
Contributor

I can confirm that the layouts are assigned based on primary role (as the member can have multiple roles and we don't have system which would allow selection of layouts for member)

However it makes sense to pick the layout from secondary role, if the primary one does not have layout assigned

intoeetive added a commit that referenced this issue Aug 19, 2022
intoeetive added a commit that referenced this issue Aug 19, 2022
…ia secondary role

(cherry picked from commit 9fa685c)
Yulyaswan pushed a commit that referenced this issue Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants