ClemBot
ClemBot copied to clipboard
Feature: Disabling Commands Server-wide
Have the ability to enable and disable certain commands server-wide.
Current thoughts:
- Command Name:
command(aliases:cmd) - Arguments:
<command name> [enable | on | disable | off](if given the command name it will let the user know if it's enabled or disabled (does not require claim)) - When enabling or disabling commands (i.e., not
!cmd <command name>) themanage_commandsclaim is required. - Only applies to a command or a group of commands. You cannot disable a sub-command, i.e.,
!tag add. - Possible idea: have another claim to bypass disabled commands? Probably named
bypass_disabled_commands? - Command Service
- Might have to rename from
CommandMetricsServicejust toCommandServiceif we don't want to add another service. - On each command run, the bot will request from the API whether or not that command is disabled. If so, the user will be notified that it's been disabled server-wide.
- Might have to rename from
- Additions to Commands Route
- [GET]
bot/commands/{GuildId}/{CommandName}- Will check against aList<string>from the API and will return a boolean,trueif enabled,falseif disabled.
- [GET]
First of all, excellent work. some thoughts
1. we still need data model design
2. we should consider adding an option to either fail silently or report that the cmmand is disabled
3. This will require additions to the docs site
4. we will need a put end and delete endpoint to add and delete mappings
overall youve covered most of the bases