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

Update resolved data structure in application command interactions #165

Closed
wants to merge 2 commits into from

Conversation

Azn9
Copy link
Member

@Azn9 Azn9 commented Mar 27, 2024

Re-created from #161

Justification: I think there was a misunderstanding in https://discord.com/developers/docs/interactions/receiving-and-responding#interaction-object-resolved-data-structure.

Partial Channel objects only have id, name, type and permissions fields. Threads will also have thread_metadata and parent_id fields.

Those fields in partial channel objects may be included, but they are not mandatory. They do have the same properties as in the channel object description.

This PR also includes the thread related fields, as they are missing.

This PR is required to fix issue #1171 in D4J.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants