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

[FIX] Client breaks when trying to deactivate a user who's the last owner of one or more room[s] #25156

Closed
wants to merge 1 commit into from

Conversation

debdutdeb
Copy link
Member

Proposed changes (including videos or screenshots)

Issue(s)

#25155

Steps to test or reproduce

Further comments

@debdutdeb
Copy link
Member Author

Ftr I don't know if it's deja vu or not, but i could swear I saw a similar patch a while ago. Or maybe it was in a different section altogether.

@ankar84
Copy link

ankar84 commented Apr 12, 2022

Ftr I don't know if it's deja vu or not, but i could swear I saw a similar patch a while ago. Or maybe it was in a different section altogether.

Actually I have the same filling!

@ankar84
Copy link

ankar84 commented Apr 12, 2022

@debdutdeb what do you think that fix could be included in some patch version like 4.6.2?

@debdutdeb
Copy link
Member Author

I think we can. @casalsgh can we?

@sidmohanty11
Copy link
Contributor

@debdutdeb I think you saw it here #24933 :)

@debdutdeb
Copy link
Member Author

Ahhhh yes! Thanks @sidmohanty11 ! I'm closing this one in favor of that.

@debdutdeb debdutdeb closed this Apr 12, 2022
@debdutdeb debdutdeb deleted the fix-last-owner-deactivation branch April 12, 2022 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants