Skip to content
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

Bot removes the reactions of itself #15

Closed
SpeckyYT opened this issue Dec 31, 2020 · 2 comments
Closed

Bot removes the reactions of itself #15

SpeckyYT opened this issue Dec 31, 2020 · 2 comments

Comments

@SpeckyYT
Copy link

I was trying this module and I found a bizzarre undocumented feature.
When clicking on the emojis of the message, the bot removes the bot's reaction instead of the user's.

@psibean
Copy link

psibean commented Jan 18, 2021

By chance was it your own bot that sent the message which then caused the pagination embed to be sent?

The only line in the code that removes ANY reaction is this:

reaction.users.remove(msg.author);

Where msg is the message you passed into the paginationEmbed function, this message should be a message that one of your users sent (and thus ran a command). The only way for the bot to remove it's own reaction instead of the users reaction is if the message you passed into paginationEmbed was actually authored by the bot.

Edit: my PR fixes this by exposing the user who reacted in the collect event, allowing them to be removed by their id instead.

@SpeckyYT
Copy link
Author

I ended up not using this module, so I'll close this issue

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

No branches or pull requests

2 participants