-
Notifications
You must be signed in to change notification settings - Fork 99
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 application command localization support. #245
base: main
Are you sure you want to change the base?
Add application command localization support. #245
Conversation
…on command create and edit methods.
…cation command object.
…thods for creating and editing of application commands.
…teractions::OptionBuilder methods.
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.
Looks good to me.
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.
Oh, sorry, I forgot to mention, but could you please include the with_localizations field on #get_application_commands
as well, if it's not too much trouble 😅
I’ll try to do that and rebase the branch soon. |
2840504
to
f90cb1f
Compare
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.
21eef87
to
67cf6a9
Compare
67cf6a9
to
2c990aa
Compare
Summary
This PR adds support for registering and editing name and description localizations of application commands and their options. I took a slightly different approach than in #190, because I don’t like the additional builder.
Added
name_localizations
anddescription_localizations
properties of application commands and their optionswith_localizations
when callingDiscordrb::Bot#get_application_commands
.