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.

Sign up for GitHub

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

Jump to bottom

Should not be able to delete SYSADMIN role #1786

Closed
successgo opened this issue Dec 16, 2021 · 1 comment · Fixed by #1788
Closed

Should not be able to delete SYSADMIN role #1786

successgo opened this issue Dec 16, 2021 · 1 comment · Fixed by #1788
Assignees
@successgo
Labels
bug core-manager
Milestone
Version 10.0.2

Comments

@successgo
Copy link
Contributor

Hi, I have noticed that SYSADMIN normalized name can not be changed to others, so if that means it's not better to delete the SYSADMIN role?

image
@tidyui
Copy link
Member

tidyui commented Dec 20, 2021

The SYSADMIN role will be checked on initialization, and if not present it will be created. In other words it can't be deleted, and it's rather a bug in the UI/API:s if it allows you to delete it.

@tidyui tidyui changed the title Can SYSADMIN role can be deleted Should not be able to delete SYSADMIN role Dec 20, 2021
@successgo successgo mentioned this issue Dec 20, 2021
Merged
@tidyui tidyui added this to the Version 10.0.2 milestone Dec 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@successgo successgo

Labels
bug core-manager
Projects
None yet
Milestone
Version 10.0.2
Development

Successfully merging a pull request may close this issue.

Hide delete button for SYSADMIN role deletion. opengdc/piranha.core
2 participants
@tidyui @successgo