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

Prevent the removal of the last admin per meeting #2528

Closed
Elblinator opened this issue Jul 18, 2024 · 2 comments · Fixed by #2542
Closed

Prevent the removal of the last admin per meeting #2528

Elblinator opened this issue Jul 18, 2024 · 2 comments · Fixed by #2542
Assignees
Labels
Milestone

Comments

@Elblinator
Copy link
Member

Current behaviour:
It is possible to have a meeting without any user as it's admin (specifically the admin group here)

This can lead to problems for the users and after the feature to lock a meeting from the inside, it can even lead to inaccessible meetings

Known ways to remove the last admin:

  • As superadmin/orgaadmin: remove/delete the last admin (possible from orga- and meeting-level)
  • As accountadmin/committeadmin: delete the last admin (possible from orga-level)
  • As "meeting" admin: remove the admin group from everyone (including yourself)

Wanted behaviour:
If someone tries to remove the last admin from a meeting this action should be blocked and thrown an error

Any meeting which already has no user in the admin group should stay like that. -> meaning no migration is needed

@Elblinator Elblinator added this to the 4.2 milestone Jul 18, 2024
@luisa-beerboom
Copy link
Member

We should also probably pay attention to being locked out of meetings

@luisa-beerboom luisa-beerboom self-assigned this Jul 26, 2024
@luisa-beerboom
Copy link
Member

It should still be possible to remove the last admin from template meetings, it should not be possible to remove template status from a template meeting without admins

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants