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.

Sign up for GitHub

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

Jump to bottom

feat: Slash Commands & Interactions [WIP?] #56

Merged
merged 11 commits into from
Dec 16, 2020
Merged

feat: Slash Commands & Interactions [WIP?] #56

merged 11 commits into from
Dec 16, 2020

Conversation

DjDeveloperr
Copy link
Member

@DjDeveloperr DjDeveloperr commented Dec 10, 2020 •
edited
Loading

This PR implements Slash Commands & Interactions as of new API update, only what's documented though.
Reference: discord/discord-api-docs#2295

Changes

  • New gateway event interactionCreate
  • Added payloads/types related to Interactions
  • Added new structure Interaction with basic handling
  • New @slash decorator

@AkiaCode AkiaCode added the enhancement ✨ New feature or request label Dec 10, 2020
@AkiaCode AkiaCode linked an issue Dec 10, 2020 that may be closed by this pull request
TODO: Make a Full Featured Gateway #11
Closed
45 tasks
@DjDeveloperr
Copy link
Member Author

I think slash-support is complete, I can un-draft this and can be merged once update is rolled out completely?

@DjDeveloperr DjDeveloperr marked this pull request as ready for review December 10, 2020 10:45
@DjDeveloperr DjDeveloperr changed the title [WIP] feat: Slash Commands feat: Slash Commands & Interactions [WIP?] Dec 10, 2020
@AkiaCode AkiaCode mentioned this pull request Dec 11, 2020
Closed
Helloyunho
Helloyunho approved these changes Dec 11, 2020
View reviewed changes
Copy link
Member

@Helloyunho Helloyunho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Can't wait to use slash command

AkiaCode
AkiaCode approved these changes Dec 13, 2020
View reviewed changes
Copy link
Member

@AkiaCode AkiaCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Helloyunho
Copy link
Member

image

Since the slash command is now in public beta, I'll merge this.

@Helloyunho Helloyunho merged commit 55ddc64 into harmonyland:main Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Helloyunho Helloyunho Helloyunho approved these changes

@AkiaCode AkiaCode AkiaCode approved these changes

Assignees

@Helloyunho Helloyunho

@DjDeveloperr DjDeveloperr

@AkiaCode AkiaCode

Labels
enhancement ✨ New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

TODO: Make a Full Featured Gateway
3 participants
@DjDeveloperr @Helloyunho @AkiaCode