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

How to deal with threads #45

Closed
hackel opened this issue May 5, 2017 · 4 comments
Closed

How to deal with threads #45

hackel opened this issue May 5, 2017 · 4 comments
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.

Comments

@hackel
Copy link

hackel commented May 5, 2017

Today I kept seeing messages in Matrix that I wasn't seeing in the actual Slack web client. I pulled out my hair for a while trying to figure out what was going on, thinking perhaps someone had blocked me personally or something, before eventually finding the messages in a thread from days earlier that I wasn't following. I understand if Matrix doesn't support threaded conversations (yet, I hope!), but it would be extremely helpful if there was some kind of indicator that a message was a part of a threaded conversation, even something as simple as a ↳ before the message or something might be good enough.

@erdnaxeli
Copy link
Contributor

Actually the webhook API the bridge is currently using is very basic. I just saw there should be a thread_ts field in the POST data, maybe it indicates the thread. This needs some testing.

@ewingrj
Copy link
Contributor

ewingrj commented Jan 26, 2018

with #66, this would only require a mapping between eventId & the msg ts. Then the msg corresponding to the thread_ts in the slack msg could be quoted or something in the matrix msg

@kiliankoe
Copy link

Some visual indication that a message is from a thread and not posted in the mainline channel would be fantastic. Otherwise I might be replying into the channel without knowing that nobody else is going to understand the context of that.

@Half-Shot
Copy link
Contributor

Unsure if there is anything left to do from the bridge side. Matrix needs to support a more seperated thread use case, but this is outside the scope here.

@jaller94 jaller94 added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements. label May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.
Projects
None yet
Development

No branches or pull requests

6 participants