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

Make Hydrogen Thread aware #552

Closed
2 tasks
germain-gg opened this issue Oct 14, 2021 · 2 comments
Closed
2 tasks

Make Hydrogen Thread aware #552

germain-gg opened this issue Oct 14, 2021 · 2 comments
Assignees
Labels
feature New feature or request

Comments

@germain-gg
Copy link
Contributor

Threads are now using a m.thread relation.

Before this is fully ready to ship without a labs flag, there are a couple of client considerations that we can take to ensure maximum backwards compatibility and continuity in conversations when threads will be available to the masses.

MSC3440 recommends two things for clients that do not fully support threads yet

  • To treat m.thread the same way that they treat m.in_reply_to
  • To change the event references from m.in_reply_to to m.thread when replying to an event that has an m.thread relation (to ensure that the message will be displayed correctly in the thread when viewed on thread's ready clients)
@bwindels bwindels added the feature New feature or request label Oct 25, 2021
@DanilaFe
Copy link
Contributor

DanilaFe commented Nov 3, 2021

If the core team doesn't have time for this, I may be able to take a look - this seems manageable.

@bwindels
Copy link
Contributor

bwindels commented Nov 4, 2021

Great @DanilaFe if you want to take a look at this. Let me know if you need any input!

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

No branches or pull requests

4 participants