Skip to content

Make modal submit message optional - #13

Open
biast12 wants to merge 1 commit into
masterfrom
modal-command-fix
Open

Make modal submit message optional#13
biast12 wants to merge 1 commit into
masterfrom
modal-command-fix

Conversation

@biast12

@biast12 biast12 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Description

Update ModalSubmitInteraction so message is Option<Message> instead of required. This matches Discord payload behavior where message is only present for modals opened from components, and absent for slash command-opened modals.

Type of Change

  • Bug fix
  • New feature
  • Breaking change

Testing

  • Run a slash command that opens a modal

Checklist

  • My code follows the style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Update `ModalSubmitInteraction` so `message` is `Option<Message>` instead of required. This matches Discord payload behavior where `message` is only present for modals opened from components, and absent for slash command-opened modals.
@biast12
biast12 requested a review from a team as a code owner July 5, 2026 22:10
@github-actions github-actions Bot added the type:bug Bug Fixes label Jul 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant