-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
FEATURE: RedirectAdministrator works as single permission #81
base: main
Are you sure you want to change the base?
FEATURE: RedirectAdministrator works as single permission #81
Conversation
Make that users with just the role RedirectAdministrator can access the backend to be able to create redirects. Fixes: neos#80
@@ -17,6 +19,10 @@ roles: | |||
|
|||
'Neos.RedirectHandler.Ui:RedirectAdministrator': | |||
privileges: | |||
- privilegeTarget: 'Neos.Neos:Backend.GeneralAccess' | |||
permission: GRANT | |||
- privilegeTarget: 'Neos.RedirectHandler.Ui:Backend.Module.Management' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not simply use Neos.Neos:Backend.Module.Management
here instead of a new custom privilege target?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried that but that did not work 🤷♂️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happened when you did?
Did you also adjust the redirect after login setting for Neos?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend using Neos.Neos:Backend.Module.Management
and Neos.Neos:Backend.GeneralAccess
and that should work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should use the default privileges instead of adding a new one.
Make that users with just the role RedirectAdministrator can access the backend to be able to create redirects.
redirect-admin.mp4
Fixes: #80