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

fix(interaction_metadata): document command_type #7383

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ImRodry
Copy link
Contributor

@ImRodry ImRodry commented Feb 7, 2025

This PR documents the command_type key that is sent on the interaction_metadata object of a Message. Not sure why this wasn't documented as the PR that added the object (#6723) did not have any discussion on it. I purposely left out the name property as a previous PR adding this was rejected (#6734) but I don't see anything against this property. It would be helpful to document both in order to be able to fully stop using interaction in favor of interaction_metadata

@ImRodry ImRodry requested a review from a team as a code owner February 7, 2025 19:27
@ImRodry ImRodry requested review from JustinBeckwith and removed request for a team February 7, 2025 19:27
@ImRodry ImRodry closed this Feb 7, 2025
@ImRodry ImRodry changed the title fix(interaction_metadata): document name and command_type fix(interaction_metadata): document command_type Feb 7, 2025
@ImRodry
Copy link
Contributor Author

ImRodry commented Feb 7, 2025

Removed name, reopening to get feedback on command_type

@ImRodry ImRodry reopened this Feb 7, 2025
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.

1 participant