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

Admins should be able to make other people admins, should not be able to delete other admins #544

Closed
1 of 4 tasks
imnasnainaec opened this issue Jul 23, 2020 · 1 comment
Closed
1 of 4 tasks

Admins should be able to make other people admins, should not be able to delete other admins #544

imnasnainaec opened this issue Jul 23, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@imnasnainaec
Copy link
Collaborator

imnasnainaec commented Jul 23, 2020 •
edited
Loading

In SiteSettings, in UserManagement, it might be convenient to add or remove administrative status on another user (and possibly have a higher owner status that cannot be revoked by mere administrators).

  • Admins make other people admins
  • Admins remove admin status of other admins

Currently admin users can delete any user except themselves from in SIteSettings. Deletion of other administrators shouldn't be allowed.

  • Prevent deleting admins (frontend)
  • Prevent deleting admins (backend)
@imnasnainaec imnasnainaec added the enhancement New feature or request label Jul 23, 2020
@imnasnainaec imnasnainaec mentioned this issue Sep 8, 2020
Closed
2 tasks
@imnasnainaec imnasnainaec mentioned this issue Dec 22, 2020
Merged
imnasnainaec added a commit that referenced this issue Dec 31, 2020
@imnasnainaec
Accounts and settings: bug fixes and User Guide fill-in. (#889)
9a28ce2 
Fill in content for Account, Project, and Admin pages in the User Guide.
[SiteSettings] Disallow admins deleting other admins (part of #544).
[AvatarUpload] Allow any image type, not just .jpg.
[UserSettings] Add feedback on SAVE, and don't allow saving with taken email address (resolves #888; more to do in #894).
@imnasnainaec
Copy link
Collaborator Author

Roles and permissions have been reworked since this issue was formulated, including introduction of a project Owner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@imnasnainaec