Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Threads: Deleted message still visible #5441

Closed
daniellekirkwood opened this issue Jan 25, 2022 · 1 comment · Fixed by #5822
Closed

Threads: Deleted message still visible #5441

daniellekirkwood opened this issue Jan 25, 2022 · 1 comment · Fixed by #5822
Assignees
@ismailgulek
Labels
A-Threads Message threading to simplify the Timeline T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-Labs Z-ThreadsInternalTesting Issue found while testing Threads internally Z-ThreadsP0 The first milestone for Threads

Comments

@daniellekirkwood
Copy link
Contributor

Your use case

Found during testing on January 22, 2022

When the user deleted the root message, the timeline showed "message deleted" (as expected).
However, the Thread panel (with the thread timeline) did not show "message deleted" (and should).
Also, the Thread list view was showing the original text as if it had not been deleted at all (not good).

Have you considered any alternatives?

No response

Additional context

No response
@daniellekirkwood daniellekirkwood added T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements A-Threads Message threading to simplify the Timeline Z-ThreadsP0 The first milestone for Threads Z-ThreadsInternalTesting Issue found while testing Threads internally labels Jan 25, 2022
@frakic
Copy link

frakic commented Mar 10, 2022

Found again during Community testing on Mar 3.

When I delete the root message within a thread, I can still see it in thread panel unless I go back to timeline or refresh the app.

@frakic frakic reopened this Mar 10, 2022
This was referenced Mar 15, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ismailgulek ismailgulek

Labels
A-Threads Message threading to simplify the Timeline T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-Labs Z-ThreadsInternalTesting Issue found while testing Threads internally Z-ThreadsP0 The first milestone for Threads
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Thread Timeline Fixes element-hq/element-ios
3 participants
@ismailgulek @frakic @daniellekirkwood