This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Include whether the requesting users has responded to an MSC3440 thread #11399
Labels
A-Threads
Threaded messages
T-Enhancement
New features, changes in functionality, improvements in performance, or user-facing enhancements.
Comments
clokep
added
T-Enhancement
New features, changes in functionality, improvements in performance, or user-facing enhancements.
A-Threads
Threaded messages
labels
Nov 19, 2021
This seems like it might not be super feasible since we don't have the requesting user when serializing events. We could probably pass through the user to those places, but I'm not sure if that is desirable (in terms of caching and such). What do we think of this? Relevant code is near:
|
This is not necessary, the filtering adding to MSC3440 is meant to be used for this purpose. |
We might need to do this for thread previews in the timeline. |
This was referenced Dec 14, 2021
This was referenced Dec 14, 2021
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.
Per an update to MSC3440. (Note that we're going to be consistent and not use a camel-case version as suggested there.)
The text was updated successfully, but these errors were encountered: