-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Slash command api #501
Slash command api #501
Conversation
…lugged into slash-command-api
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've decided not to support custom sections; all commands should go under a single section for Replugged.
thats just gonna make things cluttered for users ngl |
I disagree. Multiple sections for Replugged will be cluttered in its own right, with too many underutilized sections. |
make it a required to get approved as plugin to either use default section or if using custom section then should have atlleast more than 10 commands ez |
I don't know a single plugin that would use more than even 5 commands. And even if you could, it's definitely a lot better to use subcommands or choices instead. |
but still having the option in api is nice. like you can discourage people from using it but removing it completely feels wrong. also idk how to use sub commands. Edit: I might have known some plugins which can use more than 10 commands but they are kinda against tos of everything so better not state their name |
What's so difficult to understand about "no"? |
Co-authored-by: Albert Portnoy <[email protected]>
…lugged into slash-command-api
Co-authored-by: Federico Di Leo <[email protected]>
…lugged into slash-command-api
Co-authored-by: Federico Di Leo <[email protected]>
Co-authored-by: Federico Di Leo <[email protected]>
…lugged into slash-command-api
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Send it
adds support for slash commands. fixs the error in king fish's pr while adding the functionality to add custom sections and remove all commands you added with one function
edit: no more custom sections