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

Warn when the last admin tries to leave a room (especially invite-only) that the room can no longer be adminned #2855

Open
uhoreg opened this issue Jan 1, 2017 · 12 comments
Labels
O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@uhoreg
Copy link
Member

uhoreg commented Jan 1, 2017

We should probably prevent admins from leaving a room that would otherwise be non-empty

@uhoreg
Copy link
Member Author

uhoreg commented Jan 1, 2017

(apologies if this is a dup, but I couldn't find an existing issue)

This should also be enforced server-side, but preventing it on the client side would be helpful too.

@jaypatelani

This comment was marked as off-topic.

@richvdh
Copy link
Member

richvdh commented Jan 6, 2017

@jaypatelani: sounds like an unrelated issue.

@uhoreg: the admin could rejoin, so stopping them leaving seems over-strict.

@jaypatelani
Copy link

@rivhvdh if Admin makes room invited only before leaving he can't join back. Thanks

@uhoreg
Copy link
Member Author

uhoreg commented Jan 6, 2017

If they rejoin, then won't they just be a normal user?

If you don't want to stop them from leaving, then how about popping up a warning?

@richvdh
Copy link
Member

richvdh commented Jan 6, 2017

If they rejoin, then won't they just be a normal user?

no, power levels are in a separate event which persists.

@jaypatelani raises a valid point.

A warning certainly sounds sane.

@ara4n ara4n changed the title When the last admin leaves a room, the room can no longer be adminned Warn when the last admin tries to leave a room (especially invite-only) that the room can no longer be adminned Feb 19, 2017
@ara4n
Copy link
Member

ara4n commented Feb 19, 2017

Founder semantics could be a solution here too: https://github.com/matrix-org/matrix-doc/issues/652

@ara4n ara4n added T-Defect S-Tolerable Low/no impact on users P2 ui/ux labels Feb 19, 2017
@comzeradd
Copy link

I accidentally left an invite-only room, where I was the only admin. And now I can't re-join :/

@djokeefe
Copy link

djokeefe commented Mar 11, 2017 via email

@keremgocen
Copy link

what's the verdict on this one?

  • check if admin is the last one leaving the room
  • prompt with a warning message and prevent them from leaving until they promote another user to admin?

@kittykat
Copy link
Contributor

At the moment, we have this message:

Screenshot from 2022-01-17 12-27-03

I think we need the design team to confirm that that's how the dialog should look.

I don't think we should make the last person unable to leave a room. The product team needs to decide how to handle last admin leaving a room.

@kittykat kittykat removed P2 S-Tolerable Low/no impact on users Help Wanted Extra attention is needed labels Jan 17, 2022
@kittykat kittykat added O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Needs-Design X-Needs-Product More input needed from the Product team labels Jan 17, 2022
Arnei added a commit to Arnei/matrix-react-sdk that referenced this issue Oct 19, 2022
If the last room administrator leaves a room, other users cannot
gain admin privilges anymore, leaving the room in an unmoderable
state. To help in avoiding this scenario without actually preventing
an admin from leaving the room if they really want, this commit
adds a new warning message.

Attempts to help with: element-hq/element-web#2855

Signed-off-by: Arne Wilken arnepokemon@yahoo.de
@daniellekirkwood
Copy link
Contributor

I believe the PR linked above would close this issue and as Product and Design have given feedback on the PR will be removing those labels

@daniellekirkwood daniellekirkwood removed X-Needs-Design X-Needs-Product More input needed from the Product team labels May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests