-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Allow name edition in CE #7046
Allow name edition in CE #7046
Conversation
49d9920
to
56b46a5
Compare
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.
Look nice, just a remark because we can remove even more code in the back :)
Codecov Report
|
56b46a5
to
ec3147b
Compare
LGTM :) |
Signed-off-by: soupette <cyril.lpz@gmail.com>
ec3147b
to
11afd28
Compare
I think we need to do sth in case we rename the Super Admin role, the error messages that say you need to have at least one super admin won't make much sense. |
You are right @alexandrebodin, in my opinion and for consistency matters, we should prevent only the Super Admin renaming from the Admin Panel (it would still be possible in DB). |
I agree @JabStrapi |
Signed-off-by: soupette <cyril.lpz@gmail.com>
Signed-off-by: Alexandre Bodin <bodin.alex@gmail.com>
Signed-off-by: soupette <cyril.lpz@gmail.com>
71d9284
to
9ba69f6
Compare
…rapi into fix/ce-allow-role-name-edition
Signed-off-by: soupette cyril.lpz@gmail.com
Description of what you did:
Allow role name edition for CE.