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

Messages sometimes appear out of order in threads #5840

Closed
novocaine opened this issue Mar 17, 2022 · 2 comments · Fixed by matrix-org/matrix-ios-sdk#1412
Closed

Messages sometimes appear out of order in threads #5840

novocaine opened this issue Mar 17, 2022 · 2 comments · Fixed by matrix-org/matrix-ios-sdk#1412
Assignees
@ismailgulek
Labels
A-Threads Message threading to simplify the Timeline O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Labs Z-ThreadsInternalTesting Issue found while testing Threads internally

Comments

@novocaine
Copy link

Steps to reproduce

  1. Open a reply to a thread from a push notification (unclear if this is necessary to repro)

Outcome

What did you expect?

The messages to be ordered correctly

What happened instead?

The messages were out of order. In this screenshot, "do you have a screenshot of that" should appear last.

ima_c38dbfe

Your phone model

iPhone X

Operating system version

14.6

Application version

1.8.7 20220316153033

Homeserver

matrix.org

Will you send logs?

No
@novocaine novocaine added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems A-Threads Message threading to simplify the Timeline Z-ThreadsInternalTesting Issue found while testing Threads internally S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely labels Mar 17, 2022
@RiotRobot RiotRobot added this to Incoming in Issue triage Mar 17, 2022
@stefanceriu stefanceriu removed this from Incoming in Issue triage Mar 17, 2022
@novocaine
Copy link
Author

@ara4n was this possibly an instance of #5456 ?

@ismailgulek
Copy link
Contributor

@ara4n was this possibly an instance of #5456 ?

I don't think so, i've narrowed down this issue. It was due to processing of thread events in the wrong sequence.

@ismailgulek ismailgulek mentioned this issue Mar 17, 2022
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 O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Labs Z-ThreadsInternalTesting Issue found while testing Threads internally
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix thread events ordering matrix-org/matrix-ios-sdk
2 participants
@novocaine @ismailgulek