Skip to content

Conversation

FloatingMilkshake
Copy link
Collaborator

Closes #96. Adds a new mass command group (both !mass and /mass, it's a single combined command) with subcommands for ban, kick, mute, unmute, dehoist, undehoist

Includes aliases in the format used previously, like !massban, to avoid breaking what people are already familiar with! So all of the following are valid: !massban <users> [reason]; !mass ban <users> [reason]; /mass ban <users> [reason] (but not /massban <users> [reason])

The aliases (!massban) accept users separated by spaces and newlines, but commands in the mass group only accept separation with spaces. I think this has to do with how D#+ is parsing these commands so it's out of our control

This also moves any mass commands that already existed into the new MassCmds class, and moves KickCmds.KickAndLogAsync and .SafeKickAndLogAsync into the new KickHelpers class.

Also changes BanHelpers.BanSilently to require a moderator ID; for automatic bans on member join, the bot's own ID is used to match the behavior of other automatic actions.

Includes aliases in the format used previously, like !massban

Moves any mass commands that already existed into the new MassCmds class, and moves KickCmds.KickAndLogAsync and .SafeKickAndLogAsync into the new KickHelpers class

Also changes BanHelpers.BanSilently to require a moderator ID; for automatic bans on member join, the bot's own ID is used to match the behavior of other automatic actions
@FloatingMilkshake FloatingMilkshake moved this to Awaiting review in Cliptok todo Jun 16, 2025
@FloatingMilkshake
Copy link
Collaborator Author

but commands in the mass group only accept separation with spaces

This doesn't seem to be the case anymore? Not sure why / what changed

Copy link
Owner

@Erisa Erisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mass mute with mentions doesn't work, https://discord.com/channels/885928970137260093/885928971068387352/1402431290153894049

Gives ":CliptokCritical: Not accepting a massmute with a single user. Please use !mute."

@github-project-automation github-project-automation bot moved this from Awaiting review to In Progress in Cliptok todo Aug 5, 2025
@FloatingMilkshake FloatingMilkshake requested a review from Erisa August 6, 2025 16:24
@FloatingMilkshake FloatingMilkshake moved this from In Progress to Awaiting review in Cliptok todo Aug 6, 2025
# Conflicts:
#	Commands/DehoistCmds.cs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Awaiting review

Development

Successfully merging this pull request may close these issues.

Todo: Mass mute/unmute

2 participants