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

add Interaction Object for Messages #1747

Merged
merged 5 commits into from
Sep 5, 2021
Merged

add Interaction Object for Messages #1747

merged 5 commits into from
Sep 5, 2021

Conversation

topi314
Copy link
Contributor

@topi314 topi314 commented Jul 28, 2021

Pull Request Etiquette

Changes

  • Internal code
  • Library interface (affecting end-user code)
  • Documentation
  • Other: _____

Closes Issue: #1746

Description

This pr implements the message interaction object https://discord.com/developers/docs/interactions/slash-commands#message-interaction-object

@topi314
Copy link
Contributor Author

topi314 commented Jul 28, 2021

this should be all changes needed and I'll do docs once the implementation is fine

Copy link
Contributor

@Sanduhr32 Sanduhr32 left a comment

Choose a reason for hiding this comment

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

I'm unsure how important this is, so I'm just leaving this as a neutral comment

@DV8FromTheWorld
Copy link
Member

This PR needs docs to be merged. Looks okay otherwise.

@DV8FromTheWorld DV8FromTheWorld added the status: documentation needed lacks documentation, either partially or completely label Aug 2, 2021
# Conflicts:
#	src/main/java/net/dv8tion/jda/internal/entities/SystemMessage.java
Co-authored-by: Florian Spieß <business@minnced.club>
@MinnDevelopment MinnDevelopment added type: feature and removed status: documentation needed lacks documentation, either partially or completely labels Sep 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants