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
Update docs for kill command #203
Conversation
Could you share the reasoning for removing that check? How did you know it was that one that was dodgy? What issues was it causing? Do you know why it didn't perform as expected? What was the values of the variables instead? |
was just fucking around with it putting logging in different places and realised it never actually passed that block cos that check wasn't passing, tried removing it and it works fine. iirc the comparison was a Message object to an Interaction object, so it was just permanently failing. |
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.
Ah yes, I see. That's because confirmation_message
has type discord.Message | discord.Interaction
. It is important for us to perform this check so that we do not incorrectly assume confirmation because of a button press in the same channel that is wholly unrelated.
This is now dependent upon #202 |
No description provided.