-
Notifications
You must be signed in to change notification settings - Fork 37
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
Switch from text command to slash command #56
Comments
Yes , i was gonna ask for it , you should add ways to use slash module through slash commands :) |
What I propose in details to implement this feature and also to keep text commands if needed by some users: Starting from 2.2.x:
Starting from v3.x.x:
An example on how to register a slash command using Discord v12.5: |
I believe the option to use text commands should always be there in addition to slash commands, since discord is only removing access to message content for verified applications all unverified bots will have access to message content and every verified application who has been granted the intent will also. In this scenario in which the developer has access to message content it may be within your best interest to continue to provide an option for text based commands because it could be what a small server prefers, the standard for other commands the bot has or even just what they are used to. Text commands being removed from version 3 seems it may limit or confuse users who do not want to use slash commands (especially when considering message content will not be out of reach for some developers) |
Also perhaps a quicker way to keep the node requirement down to only v14 could be to install a master branch version that was created prior to the addition of the version 16 requirement (I'm using a version of v13 that functions on node 14 like this) |
Thank you for the feedback. I understand your point of view and I know that a part of developers will continue to use text commands. As the module seems to be relatively stable, the third version of it should not be out before 2022-2023, so I will support both options for now and I will see how it goes in next months. I do not want to confuse users and I would like to be as inclusive as possible. The module is designed with latest Discord guidelines and bot design practices in mind. If users are still using text commands I will leave the option as is and do not remove it completly. I have edited my last comment to take this idea into consideration.
You are talking about the possibility to enable both at the same time? Is that really useful for users?
Thank you for the suggestion, I'll check it out! I'm a bit confused because Discord.js v13 is not backwards compatible at all, so I'm afraid it will not be easy to have the same source code for both versions. I'm still working on this. |
Ah alright that sounds good The discordjs version I linked should be able to run v13 on node 14 the only inconvenience is as you would expect from a dev branch some bugs might be present, as long as no changes affect the functionality it could work. See v13.1.0 changelog for issues present within v13.0.0 and previous commits Note: Using the version I provided will also hinder ability to use context menus |
Slash command is coming soon! 👀 To avoid API rate limit errors, I have also decided to add two admin commands |
GH-56: Use slash command as the default one
GH-56: Customize option name for slash command
As Discord is evolving months after months, a new feature has been added to use slash commands instead of text commands [1]. This improves accessibility for Discord users. That's why I thinking of moving the module in that way and deprecate text commands.
Also to support users privacy, a change wiil be made next year to Discord API to limit certified bots to use message contents [2]. I think it's a good decision and I would like to do the same for the module.
Feel free to give your feedbacks about that idea 😄
[1] Discord.js article about the subject: https://blog.discord.com/slash-commands-are-here-8db0a385d9e6
[2] Modifications on Discord.js API : https://support-dev.discord.com/hc/en-us/articles/4404772028055
The text was updated successfully, but these errors were encountered: