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

Improve email notification context for threads #11976

Open
clokep opened this issue Feb 11, 2022 · 2 comments
Open

Improve email notification context for threads #11976

clokep opened this issue Feb 11, 2022 · 2 comments
Labels
A-Email-Push Email notifications 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 Feb 11, 2022

When generating email notifications Synapse fetches the event before and after the event which caused the notification to include context in the email.

Unfortunately this won't work well for threads where the "event" before and after aren't necessarily linear with what the person was replying to. We should fetch the event before and after that are in the same thread.

@clokep clokep added T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. A-Threads Threaded messages labels Feb 11, 2022
@clokep clokep self-assigned this Feb 11, 2022
@clokep clokep assigned clokep and unassigned clokep Feb 17, 2022
@daniellekirkwood
Copy link

Is this just for Email notifications?

@clokep clokep changed the title Improve notification context for threads Improve email notification context for threads Feb 17, 2022
@clokep
Copy link
Member Author

clokep commented Feb 17, 2022

Is this just for Email notifications?

Yes! I've updated the title!

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

No branches or pull requests

3 participants