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

Improvement - error-user-is-last-owner #17805

Closed
rsjr opened this issue Jun 2, 2020 · 1 comment · Fixed by #16088
Closed

Improvement - error-user-is-last-owner #17805

rsjr opened this issue Jun 2, 2020 · 1 comment · Fixed by #16088

Comments

@rsjr
Copy link

rsjr commented Jun 2, 2020

Description:

Currently, it is not possible to delete an user when he is the last owner of a room. This behavior is set on deleteUser.js.

Steps to reproduce:

  1. End user is the only owner of a room
  2. Admin tries to delete him (via API or Users panel)
    3.Error is triggered

Expected behavior:

The error explanation itself is not clear, and sometimes an admin should have an 'override' to delete the user anyway.

Two suggestions in order to improve this scenario:

  1. Add a special parameter in the API method call so the admin can delete the user even if he is the only owner of a room. The logic should add the oldest room user as the owner (or add the possibility for the admin himself to select who should be the owner)

OR

  1. At least improve the error message and display the rooms which the user is the owner so the admin can go there and adjust it manually instead of having to look for it,.

Server Setup Information:

  • Version of Rocket.Chat Server: 3.3.0-develop
@sampaiodiego
Copy link
Member

the PR #16088 closes this issue

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

Successfully merging a pull request may close this issue.

2 participants