search

Wylx-Bot / Wylx

Discord Bot
7 stars 2 forks source link

Make disabling packages better / ;help #97

Open VVill-ga opened 2 years ago

VVill-ga commented 2 years ago

Right now the command to disable a package goes something like this: ;package TTRPGPackage false. I feel like the second "package" is unnecesary, and so is the boolean if the command is just used to toggle it on and off. If the ttrpg package is on, I would like to be able to say ;package ttrpg to disable it.

For context, the server I'd like to use this bot on doesn't require many of the packages included here, and I think the ;help command would be nicer if it only showed what commands are available to use if the person who sent the command is not an administrator. So for example, a normal user should not be able to see the server settings and utils package ever, and if I disable the ttrpg package, none of those commands should show up when they run ;help either. I think its smart for admins to still see disabled packages when they run ;help though.

joshuamb419 commented 2 years ago

good ideas, help is something that needs redone. what do you think about having the currently disabled packages hidden and then having an arg for help that would reveal them? Current reason the second package exists is because currently it just rips the class name of the package thats holding it, so, its a naming thing in code that needs to change or get scrubbed from the names when they get used.

joshuamb419 commented 2 years ago

@1Revenger1 maybe look at this when you rewrite help. Ill take care of the making package/command enabling/disabling more streamlined but I think showing non-admins only the enabled commands is a good idea