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

Remove flagger? #137

Open
kmarekspartz opened this issue Aug 30, 2017 · 3 comments
Open

Remove flagger? #137

kmarekspartz opened this issue Aug 30, 2017 · 3 comments

Comments

@kmarekspartz
Copy link
Contributor

It's underutilized and a frequent source of bugs. Can we get rid of it?

@TheConnMan
Copy link
Contributor

Flagging based on reactions feels like a more real-time operation, not nightly. I swear I've seen a bot out there that does what the flagger does in real-time (though not based on chatted in rules). Might make sense to either use one of those bots or build a different bot using the Slack Events API.

@kmarekspartz
Copy link
Contributor Author

Yeah https://reacji-channeler.builtbyslack.com is the real-time one.

Some people prefer the async reminder as a spaced repetition sort of thing. Additionally, the flagger allows arbitrary thresholds, e.g. requiring two or more reacji before notifying.

In the Rands Slack, I have converted the single reacji cases to use reacji channeler, but a few rules still remain.

@TheConnMan
Copy link
Contributor

That's the one. Makes sense, especially for rules which take multiple reactions.

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

No branches or pull requests

2 participants