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

Thread summary does not take into account edited events #11973

Closed
clokep opened this issue Feb 11, 2022 · 0 comments · Fixed by #11992
Closed

Thread summary does not take into account edited events #11973

clokep opened this issue Feb 11, 2022 · 0 comments · Fixed by #11992
Assignees
@clokep
Labels
A-Threads Threaded messages S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.

Comments

@clokep
Copy link
Member

clokep commented Feb 11, 2022

If the latest event in a thread is edited, the edit is not taken into account when calculating the thread summary.

This would happen if you have:

  1. A root event (A).
  2. A threaded event (B).
  3. An edit to B (C).

The summary shows B as the latest event instead of C.
@clokep clokep added S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. A-Threads Threaded messages labels Feb 11, 2022
@clokep clokep self-assigned this Feb 11, 2022
@clokep clokep mentioned this issue Feb 14, 2022
Merged
@clokep clokep mentioned this issue Mar 23, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@clokep clokep

Labels
A-Threads Threaded messages S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix a bug with thread summaries when the latest event is edited matrix-org/synapse
1 participant
@clokep