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

{URGENT} can self-star pls fix >:((((((( #1

Open
qixils opened this issue Nov 1, 2017 · 6 comments
Open

{URGENT} can self-star pls fix >:((((((( #1

qixils opened this issue Nov 1, 2017 · 6 comments
Labels

Comments

@qixils
Copy link
Member

qixils commented Nov 1, 2017

>:((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((

@Bottersnike
Copy link
Contributor

I'm afraid this is only doable if we can guarantee 24/7/365 up-time which we can't so I find it's safer to allow self-staring but have a functional bot as opposed to the other way around. Feel free to re-open if you have a suggestion as to how to go about this without just watching the events and then creating our own emoji counter that ignores the op.

@adamthedog
Copy link
Member

Why is this only doable with constant uptime? Just compare the starrer's ID with that of the message author ID and if they're the same, pass.

@Bottersnike
Copy link
Contributor

That stops them being star 3, but if they were star 1, then the 3rd star wouldn't be by them and we use .count which would return 3. The alternative is monitoring the events and then keeping our own reactions DB but then if there's any down-time that DB gets out of sync and will be missing some reactions from it.

@adamthedog
Copy link
Member

In :meth:.action, you could make use of the users iterator and .flatten() to check if the message author is in the list of reactors. You'd also have to make sure the reaction they gave was not a star. I may be getting this all wrong though, it's 3 in the morning.

@Bottersnike
Copy link
Contributor

https://github.com/Rapptz/RoboDanny/blob/rewrite/cogs/stars.py#L386 That appears to be how Danny's doing it as well (keeping a DB of the reactions)

@qixils
Copy link
Member Author

qixils commented Nov 20, 2017

apparently this bug is back/wasn't fixed?

@qixils qixils reopened this Nov 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants