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

Figure out how to handle reacts from Slack correctly #25

Open
jvperrin opened this issue Sep 7, 2017 · 2 comments
Open

Figure out how to handle reacts from Slack correctly #25

jvperrin opened this issue Sep 7, 2017 · 2 comments

Comments

@jvperrin
Copy link
Member

jvperrin commented Sep 7, 2017

Not sure what would be best for this, since it needs context like threads but also seems like it doesn't warrant an individual message for each react, since there can be a lot of them. That being said, not mirroring them loses out on some context, although admittedly it's not a huge amount generally. Sometimes reacts are used to indicate that someone has acknowledged a message without a full reply, so it's useful for that.

@kkuehlz
Copy link
Member

kkuehlz commented Jul 9, 2018

we could have bridgebot pm a user about a reaction? could get annoying for some. We might want this to be an optional feature... i.e pm the bridge to register for reacts. then that requires some kind of persistent storage though.

@jvperrin
Copy link
Member Author

jvperrin commented Jul 9, 2018

PMing on reactions seems reasonable, that's about the only way I could think of doing this, but I agree with you that it could be annoying for some or spammy. In Slack it's not really sent as a notification at all, although it does show a little icon in the top corner I believe (this is probably on purpose so that it's not annoying or spammy). Maybe we should just ignore trying to get the reactions to mirror since they aren't really that important? Compared to PMs or private channels for instance, I think those would be much more useful to mirror.

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