Skip to content

fix: defer report modal reply#109

Merged
Owen1212055 merged 1 commit intoMCDiamondFire:masterfrom
dewsmith0:master
Mar 9, 2026
Merged

fix: defer report modal reply#109
Owen1212055 merged 1 commit intoMCDiamondFire:masterfrom
dewsmith0:master

Conversation

@dewsmith0
Copy link
Contributor

@dewsmith0 dewsmith0 commented Mar 9, 2026

This changes the report modal to defer the interaction while the bot uploads the files to the mod channel, preventing Discord from timing out the interaction when uploading large files and leading users to submit the report multiple times.

Tested locally, bot showed [HelpBot] is thinking... while it was uploading, sent the files, and successfully replied (ephemerally) to the interaction.

Copy link

@zBinFinn zBinFinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I don't have the relevant permissions to merge this yet I'll get back to you when I do

@Owen1212055 Owen1212055 merged commit 1415517 into MCDiamondFire:master Mar 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants