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

Tell the timeline window when an event is removed (fixes element-web/issues/26498) #11836

Closed

Conversation

andybalaam
Copy link
Contributor

@andybalaam andybalaam commented Nov 8, 2023

Prevents element-hq/element-web#26498 (Can't remove a reaction from a thread) from happening, even when matrix-org/matrix-js-sdk#3798 (Move deleted messages into main timeline) is applied.


Here's what your changelog entry will look like:

🐛 Bug Fixes

  • Tell the timeline window when an event is removed (fixes element-web/issues/26498) (#11836). Contributed by @andybalaam.

@andybalaam andybalaam added the T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems label Nov 8, 2023
@andybalaam andybalaam requested a review from a team as a code owner November 8, 2023 11:49
Prevents element-hq/element-web#26498 (Can't
remove a reaction from a thread) from happening, even when
matrix-org/matrix-js-sdk#3798 (Move deleted
messages into main timeline) is applied.
@andybalaam andybalaam force-pushed the andybalaam/tell-window-when-event-is-removed branch from 3fe20b7 to 2965614 Compare November 8, 2023 11:50
Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doing it here means it'll be broken for all other matrix-js-sdk Applications unless they mimic this behaviour, this is not a good fix.

@andybalaam
Copy link
Contributor Author

Re-done in matrix-org/matrix-js-sdk#3862 in a way that should work automatically for any user of the TimelineWindow.

@andybalaam andybalaam closed this Nov 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants