Add ban user by id command /banid
#4411
Conversation
| @@ -2829,6 +2829,54 @@ void CommandController::initialize(Settings &, Paths &paths) | |||
| return ""; | |||
| }); | |||
|
|
|||
| this->registerCommand("/banid", [formatBanTimeoutError]( | |||
There was a problem hiding this comment.
Open to other suggestions for the command name
There was a problem hiding this comment.
this name is fine for me
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -2829,6 +2829,54 @@ void CommandController::initialize(Settings &, Paths &paths) | |||
| return ""; | |||
| }); | |||
|
|
|||
| this->registerCommand("/banid", [formatBanTimeoutError]( | |||
There was a problem hiding this comment.
this name is fine for me
Co-authored-by: Felanbird <41973452+Felanbird@users.noreply.github.com>
Maybe special syntax could be made so all |
Co-authored-by: pajlada <rasmus.karlsson@pajlada.com>
If relevant use-cases come up in the future, we'll figure out a uniform solution for this that can be applied to all commands but for now we'll go with this simple one that has an actively request use-case. |
pajlada
changed the title
/banid
Pull request checklist:
CHANGELOG.mdwas updated, if applicableDescription
With the
/bancommand migrated to Helix, we are no longer able to ban suspended or deactivated accounts because helixgetUsersis not able to resolve theuser_idassociated with the login name. However, if a moderator already knows theuser_id, helixbanUseris able to ban the account, even if suspended or deactivated. Thus, this PR adds a/banidcommand to allow banning users by id.Note: similar concerns apply for timeout/unban, but are not covered by this PR (and are not as important imo)