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

Threads: /messages filter to exclude thread responses & relations #240

Open
Tracked by #187
t3chguy opened this issue Apr 26, 2022 · 5 comments
Open
Tracked by #187

Threads: /messages filter to exclude thread responses & relations #240

t3chguy opened this issue Apr 26, 2022 · 5 comments

Comments

@t3chguy
Copy link
Member

t3chguy commented Apr 26, 2022

Is your feature request related to a problem? Please describe.
Right now, scrolling up in a room which heavily uses threads is really slow and feels broken as most /messages results are not for the main timeline and instead are events from threads.

Describe the solution you'd like
/messages filter which lets the client paginate the main timeline where the responses only include events for that timeline

Describe alternatives you've considered
Making the server go faster so this is less painful but it is still quite wasteful of data and worst case could still result in many pages of content which cannot be rendered in that timeline.

@clokep
Copy link
Contributor

clokep commented Oct 11, 2022

MSC3874 was written to solve this.

@daniellekirkwood
Copy link
Contributor

We're hoping to make this happen quickly after removing Threads from Beta.
Adding to the Web board so we don't lose it...

@daniellekirkwood
Copy link
Contributor

Removing from the WAT board as it's not currently planned. We will add it back when we loop to Threads as a bug theme

@daniellekirkwood
Copy link
Contributor

@clokep could you help me understand what work and which team would be needed here? Let's DM!

@clokep
Copy link
Contributor

clokep commented Sep 5, 2023

@clokep could you help me understand what work and which team would be needed here? Let's DM!

  • It looks like there's no open threads on the MSC. ✅
  • I don't think this MSC received review from any of the SCT yet. ❌
  • There's a Synapse implementation. ✅
  • The Synapse implementation seems to still match the MSC. ✅
  • There's no client implementation. ❌

@daniellekirkwood In summary, I think this is with a client team to implement in order to ensure it fixes a problem. I can enable it on an experimental server if needed.

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

5 participants