Skip to content
This repository has been archived by the owner on Dec 24, 2021. It is now read-only.

Instead of testing trigger message, test latest response #367

Merged
merged 4 commits into from
Jan 30, 2021

Conversation

dragonfire535
Copy link
Contributor

@dragonfire535 dragonfire535 commented Jan 30, 2021

https://discord.js.org/#/docs/commando/master/class/Argument?scrollTo=validate

The msg parameter is listed as the message that triggered the command, but this breaks functionality if you need to check the message received from the prompt instead. Such as if I wanted to check if there were attachments in the message, but I don't want to check the original message, I want the new prompted message.

This seems like an oversight and should change to pass the new message rather than the original one.

This seems like the solution to a long-time bug I've had where it won't take attachment images after prompt:
https://cdn.discordapp.com/attachments/316734873358434304/805111520807223326/unknown.png

This has been tested and no adverse reaction have occurred when used on Xiao.

@dragonfire535 dragonfire535 merged commit 614dd95 into master Jan 30, 2021
@dragonfire535 dragonfire535 deleted the dragonfire535-patch-1 branch January 30, 2021 21:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant