-
-
Notifications
You must be signed in to change notification settings - Fork 7k
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 admins being able to suspend their instance actor #14567
Fix admins being able to suspend their instance actor #14567
Conversation
be81b8b
to
6c0ffb4
Compare
6f83370
to
4b4ca8a
Compare
ping @Gargron |
Back-end changes are good but what if instead of adding a flash message which made this account not show up in accounts list, make its name unclickable in places where it appears, and made it impossible to view its account page? |
hm… it's possibly a lot of places, though… |
4b4ca8a
to
c744fd5
Compare
Rebased and made it so the instance actor doesn't show up in the accounts list in moderation view. It could show up in other parts but I think that's fine. Kept the flash message in case someone stumbles upon it. |
c744fd5
to
9ae10ae
Compare
9ae10ae
to
089368c
Compare
I'm not sure how, but I've seen a couple admins manage to suspend their instance actor (which cause federation issues).
This PR:
Account#suspend!
do nothing on the instance actorAccount#suspend?
return false no matter what on the instance actor