-
Notifications
You must be signed in to change notification settings - Fork 82
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 support for modals #528
Conversation
You can already receive modals via the gateway and send modals via REST
} | ||
|
||
public suspend inline fun createIntAutoCompleteInteractionResponse( | ||
public inline suspend fun createIntAutoCompleteInteractionResponse( |
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.
it's suspend inline fun
everywhere else, why change it?
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.
Honestly I'm not sure why I shuffled that, I think I removed the inline
by mistake so IDEA was complaining, so when I readded them I added them in a non-canonical order
…s and text inputs sharing the same field name
) : DiscordComponent() | ||
|
||
@Serializable | ||
public data class ActionRowComponent( |
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.
DiscordActionRowComponent
to avoid confusion with core naming (applies to all DiscordComponent
subtypes)
I already implemented the classes in the core module and now Kord compiles, yay! I haven't implemented the events because I'm not really knowledgeable with the (Or maybe I will do it myself later) |
merged to #531 |
You can already receive modals via the gateway and send modals via REST
#527