Skip to content
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 support #14

Closed
jchristgit opened this issue Aug 12, 2021 · 4 comments · Fixed by #16
Closed

Slash command support #14

jchristgit opened this issue Aug 12, 2021 · 4 comments · Fixed by #16
Assignees
Labels
enhancement New feature or request

Comments

@jchristgit
Copy link
Owner

Per discussion in #1 with @Bentheburrito.

I think Slash Commands are different enough to warrant their own behaviour imo. I have a rudimentary slash command handler on another bot that I could clean up and use as a starting point? Also, should I open a separate issue for further discussion?

My current idea:

  • Implement a new Nosedrum.SlashCommand behaviour.
  • Implement an interaction handler (possibly some kind of merge between Nosedrum.Invoker and Nosedrum.Storage, since the Invoker is less useful here) to dispatch, store and manage commands.
@jchristgit jchristgit added the enhancement New feature or request label Aug 12, 2021
@Bentheburrito
Copy link
Contributor

I noticed Discord recently changed terminology to "Application Commands" https://discord.com/developers/docs/interactions/application-commands - which include User/Message commands in addition to Slash Commands. Are these two new types of commands something you see Nosedrum supporting?

@jchristgit
Copy link
Owner Author

Hmm, from a quick glance over it, it looks like these are pretty much just built on top of the existing slash commands? Interesting, I haven't seen these before. Yeah, I think we should possibly name it something like Nosedrum.ApplicationCommand then, and integrate with that

@Bentheburrito
Copy link
Contributor

Cool! I'm happy to get started on this then :)

@jchristgit
Copy link
Owner Author

Awesome, I'll assign you then, Thank you!

@jchristgit jchristgit linked a pull request Nov 26, 2021 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants