-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat: interactions #5448
feat: interactions #5448
Conversation
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.
Just some minor nits
Co-authored-by: izexi <[email protected]>
Co-authored-by: izexi <[email protected]>
Co-authored-by: Sugden <[email protected]>
Co-authored-by: Sugden <[email protected]>
Curious — what still needs to be done to get slash commands in, and how can outside developers in the community help? Really looking forward to being able to use discord.js for slash commands, and would be interested in helping out |
There's nothing specific that needs to be done that wouldn't already be in the comments here. The PR is pending reviews and approvals from the maintainers. We've also had to keep up with ongoing changes from Discord - the API has only reached a state of stability quite recently. |
🎉 |
Wohoooo 🎉 |
Any eta on when this will be part of a release? |
Soon:tm:, we'll go thru all the |
If someone wanted to use it as I do npm i discordjs/discord.js#master works just fine |
you can just use |
Please describe the changes this PR makes and why it should be merged:
Attempt #2! Most of the logic is taken from #5106, but with a giant refactor, most notably the interaction client class being removed and an application command manager class being added. To see a quick example of how this branch is structured, have a look here.
Additionally, this PR also adds the new
USE_APPLICATION_COMMANDS
permission, theAPPLICATION_COMMAND
message type and theEPHEMERAL
andLOADING
message flags.Status and versioning classification: