Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Notifications for threads #12550

Closed
11 tasks done
clokep opened this issue Apr 26, 2022 · 2 comments · Fixed by #14050
Closed
11 tasks done

Notifications for threads #12550

clokep opened this issue Apr 26, 2022 · 2 comments · Fixed by #14050
Assignees
Labels
A-Threads Threaded messages T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.

Comments

@clokep
Copy link
Member

clokep commented Apr 26, 2022

MSC3771 and MSC3773 make changes to the receipt and notifications architecture for handling threads.

  • Develop proof of concept for MSC3771 and MSC3773 together
  • MSC3773: Notifications for threads
    • Update the notifications schema to handle a thread field.
    • Update the push rule handling code to attach thread to notifications.
    • Update the notification rotation code to handle unique threads.
    • Update the sync code to return notifications separately for each thread.
  • MSC3771: Read receipts for threads
    • Update the /receipt servlet handling for thread.
    • Update the receipt schema to handle thread (and properly handle it when not provided).
    • Update the fetching of receipts to handle unique threads.
    • Update the sync code to return the thread of each receipt.
@clokep clokep added T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. A-Threads Threaded messages labels Apr 26, 2022
@clokep clokep self-assigned this Apr 26, 2022
@clokep clokep changed the title Implement MSC3771: Read receipts for threads Notifications for threads Jul 25, 2022
@clokep
Copy link
Member Author

clokep commented Jul 25, 2022

The current proof of concept for this is on the clokep/thread-read-receipts-poc branch.

@clokep
Copy link
Member Author

clokep commented Oct 4, 2022

I think the last task here is to support stable identifiers, which would be needed for #13954.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Threads Threaded messages T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant