-
Notifications
You must be signed in to change notification settings - Fork 22
[Feature]: Move most Command Permissions booleans to each permission entry rather than globally #208
Description
🚀 Description
Currently, the mod puts fields such as "keepOriginal" and "ignoredCommands" into the global Chat config. However, this proposal admits that certain users might be more active than others when using commands, so by utilizing the role systems of both Minecraft mods and Discord, we have an opportunity to re-architect how command mirroring is done.
The suggestion is to move such flags to each permission entry in the linked commands system instead. That way each permission level and or discord role can have different settings for how the bot behaves on the discord (such as to hide the fact a moderator executed a command), or if the server will mirror output of those commands at all (such as to hide exactly what weather state that role set it to)
Admittedly, this may complicate converting older versions of the config to this one. I suggest a shotgun approach, where the old settings of the flags, once upgraded, just copy into every entry in the current command permissions array.
💡 Use Case
This was reported by a user on our discord interested in hiding moderator command behaviour, but to maintain command behaviour by general users, so that the annoyance of setting weather and time by a moderator could be reduced.