-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Document channel_types in ApplicationCommandOption #3825
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
| required? | boolean | if the parameter is required or optional--default `false` | | ||
| choices? | array of [application command option choice](#DOCS_INTERACTIONS_APPLICATION_COMMANDS/application-command-object-application-command-option-choice-structure) | choices for `STRING`, `INTEGER`, and `NUMBER` types for the user to pick from, max 25 | | ||
| options? | array of [application command option](#DOCS_INTERACTIONS_APPLICATION_COMMANDS/application-command-object-application-command-option-structure) | if the option is a subcommand or subcommand group type, this nested options will be the parameters | | ||
| channel_types? | array of [channel types](#DOCS_RESOURCES_CHANNEL/channel-object-channel-types) | if the option is a channel type, the channels shown will be restricted to these types | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are empty arrays treated weirdly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you provide an empty array it will just use all channel types (at least from my testing that's what happened)
it works in my lib when I set it null and ignore null on transmit. |
Is this validated by the api or is it just client side? |
validated by the API (it's already live) |
#3825 (comment) is this intended? |
yes |
* discord/discord-api-docs#3825 * discordjs/discord.js#6640 * possible because of 9bd1d25 * i should really fix these typings
No description provided.