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

Load the thread list using server-side sorting and pagination #195

Closed
4 tasks done
Tracked by #187
daniellekirkwood opened this issue Mar 5, 2022 · 3 comments
Closed
4 tasks done
Tracked by #187

Load the thread list using server-side sorting and pagination #195

daniellekirkwood opened this issue Mar 5, 2022 · 3 comments
Assignees
@clokep @gileluard @fedrunov
Labels
A-Threads T-Enhancement X-Spec-Changes Z-Labs

Comments

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Mar 5, 2022 •
edited
Loading

We currently load the entire thread list without pagination and sort it on the client so that we can sort by last-replied-to.

Add support for that sort to the server query and load the list using pagination.
@daniellekirkwood daniellekirkwood self-assigned this Mar 5, 2022
@github-actions github-actions bot added the Z-Labs label Mar 5, 2022
@daniellekirkwood daniellekirkwood mentioned this issue Mar 5, 2022
Closed
@novocaine novocaine removed the Z-Labs label Apr 22, 2022
@novocaine novocaine changed the title Order Thread list by most recent event (not root message date/time) Load the thread list using server-side sorting and pagination Apr 22, 2022
@novocaine novocaine assigned clokep and unassigned daniellekirkwood Apr 22, 2022
@clokep
Copy link
Contributor

clokep commented Jul 27, 2022

MSC3856: Threads List API was written to solve this issue.

@justjanne justjanne self-assigned this Aug 4, 2022
This was referenced Sep 27, 2022
Closed
Closed
@clokep
Copy link
Contributor

clokep commented Sep 27, 2022

See #152 for an earlier incarnation of this issue.

@justjanne justjanne removed their assignment Oct 7, 2022
@daniellekirkwood
Copy link
Contributor Author

All sub-tasks are marked as closed so I'm closing this issue...

@clokep clokep mentioned this issue Jan 24, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@clokep clokep

@gileluard gileluard

@fedrunov fedrunov

Labels
A-Threads T-Enhancement X-Spec-Changes Z-Labs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@novocaine @clokep @germain-gg @justjanne @gileluard @fedrunov @daniellekirkwood