-
Notifications
You must be signed in to change notification settings - Fork 79
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
Select menu types #707
Select menu types #707
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, very excited for some of these menu types 👀
Do you have unpushed changes for this? Since you already marked the comments as resolved. |
Yes! I'll try to make commit later today |
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, I just have a few last things I changed and just committed. Can you take a look at them? Then we should be good to go.
Sure |
It's ok for me. |
Alright, thanks! |
The deprecations for SelectMenuBuilder.options and SelectMenuComponent.options were propagated to the overrides in StringSelectBuilder and StringSelectComponent when a user of Kord didn't have progressive mode enabled. To fix this, extensions are provided instead. The changes in the public API are binary incompatible compared to #707 but not compared to 0.8.0-M17 so only snapshot users can be affected. see #707, https://youtrack.jetbrains.com/issue/KT-47902 and https://discord.com/channels/556525343595298817/587324906702766226/1042799716745879643
The deprecations for SelectMenuBuilder.options and SelectMenuComponent.options were propagated to the overrides in StringSelectBuilder and StringSelectComponent when a user of Kord didn't have progressive mode enabled. To fix this, extensions are provided instead. The changes in the public API are binary incompatible compared to #707 but not compared to 0.8.0-M17 so only snapshot users can be affected. see #707, https://youtrack.jetbrains.com/issue/KT-47902 and https://discord.com/channels/556525343595298817/587324906702766226/1042799716745879643
Major API changes are:
ComponentType.SelectMenu
toComponentType.StringSelect
ComponentType
s:UserSelect
,RoleSelect
,MentionableSelect
andChannelSelect
SelectMenuComponent
subtype for each select menu typeActionRowBuilder
see discord/discord-api-docs#5543 and discord/discord-api-docs#5544