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

Support threads with m.thread relation (MSC3440) #634

Closed
2 tasks done
jaller94 opened this issue Oct 26, 2021 · 4 comments · Fixed by #668
Closed
2 tasks done

Support threads with m.thread relation (MSC3440) #634

jaller94 opened this issue Oct 26, 2021 · 4 comments · Fixed by #668
Assignees
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.

Comments

@jaller94
Copy link
Contributor

jaller94 commented Oct 26, 2021

Threads are being added to the Matrix protocol soon. Because Slack already supports threads in its protocol and interface, it would be good to bridge that correctly. Currently, Slack threads are bridged as replies.

Related issues:

Definition of Done

  • Slack threads are bridged to Matrix as m.thread relations – removing the reply relation we currently set.
  • Matrix threads are bridged to Slack as threads.
@abmantis
Copy link

abmantis commented Dec 6, 2021

What would be the behavior in clients that do not implement threads yet? Will they still fallback to replies?

@jaller94
Copy link
Contributor Author

jaller94 commented Dec 7, 2021

The fallback will like be that the messages are shown without context in the main thread – not as replies.
This is unlikely to affect the plan of implementing this in a bridge once it's in the stable Matrix Spec.

@abmantis
Copy link

abmantis commented Dec 7, 2021

Will this be only implemented when this gets into the stable spec? :(

@Half-Shot
Copy link
Contributor

We're planning to implement this sooner.

@jaller94 jaller94 self-assigned this Feb 6, 2022
@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

Successfully merging a pull request may close this issue.

3 participants