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.

Sign up for GitHub

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

Jump to bottom

Make Element iOS Thread aware #5007

Closed
2 tasks
germain-gg opened this issue Oct 14, 2021 · 0 comments · Fixed by matrix-org/matrix-ios-kit#940
Closed
2 tasks

Make Element iOS Thread aware #5007

germain-gg opened this issue Oct 14, 2021 · 0 comments · Fixed by matrix-org/matrix-ios-kit#940
Assignees
@ismailgulek
Labels
A-Threads Message threading to simplify the Timeline T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-ThreadsP0 The first milestone for Threads

Comments

@germain-gg
Copy link
Contributor

Your use case

What would you like to do?

To take a few steps towards understanding Threads in Android before a full implementation

Why would you like to do it?

To ensure maximum backwards compatibility and continuity in conversations when threads will be available to the masses.

How would you like to achieve it?

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)

Have you considered any alternatives?

No response

Additional context

Threads might be using a m.thread relation over m.in_reply_to
@germain-gg germain-gg added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Oct 14, 2021
@germain-gg germain-gg mentioned this issue Oct 14, 2021
Closed
@ismailgulek ismailgulek self-assigned this Oct 14, 2021
@daniellekirkwood daniellekirkwood mentioned this issue Oct 14, 2021
Closed
@daniellekirkwood daniellekirkwood added A-Threads Message threading to simplify the Timeline Z-ThreadsP0 The first milestone for Threads labels Oct 14, 2021
This was referenced Oct 22, 2021
Merged
Merged
@ismailgulek ismailgulek mentioned this issue Nov 11, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ismailgulek ismailgulek

Labels
A-Threads Message threading to simplify the Timeline T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-ThreadsP0 The first milestone for Threads
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Threads P0 matrix-org/matrix-ios-kit
3 participants
@germain-gg @ismailgulek @daniellekirkwood