Skip to content
This repository has been archived by the owner on Sep 23, 2022. It is now read-only.

[RoleManagement] Suggestion | Roleset #209

Closed
Menteroth opened this issue Feb 23, 2020 · 2 comments
Closed

[RoleManagement] Suggestion | Roleset #209

Menteroth opened this issue Feb 23, 2020 · 2 comments

Comments

@Menteroth
Copy link

Describe the solution you'd like
When setting roles exclusive to each other it would be nice to have the bot remove the reaction your first chose if you react with something else later. So that if you need/want to go back to the initial reaction you don't have to unreact before reacting again. Plus it also shows the correct number of reactions compared to roles added to members.

@mikeshardmind
Copy link
Owner

Won't be added

@mikeshardmind
Copy link
Owner

As there's new attention on this since supporting again...

The "won't be added" bit was a glib quick response that didn't deal with explaining why this change is not easily compatible with the cog.

Discord does not send information about who removed the reaction in the same payload as the reaction events (it is inconsistently available from the audit log) This also increases the rate limit consumption, the number of reactions matching is also not something this achieves, as discord does not send the full reaction info to the desktop or mobile clients.

This could lead to conditions that are messy to deal with properly and I cannot suggest this, despite agreeing that in a perfect scenario, if the roles were set as exclusive, this would be done.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants