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.

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

Messages can appear out of order if /sync is delayed but message PUTs are 200ing #1934

Open
ara4n opened this issue Feb 6, 2017 · 2 comments
Labels
A-Timeline P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Feb 6, 2017

If you send a bunch of messages on a slow connection, they stack up in grey. It's possible they then go black, as you receive 200s for them. However, if you then receive /sync response the syncs can end up before the queued messages, making them appear out of order. At least, i think that's what's going on.

It's closely related to element-hq/element-web#639.

I've seen it a bunch and we just got a report at https://matrix.to/#/!DgvjtOljKujDBrxyHk:matrix.org/%2414864078871816754pzfgA:matrix.org of it.

@axelsimon
Copy link

Can expand a bit. The experience is that message end up appearing earlier in the conversation (which I think is ok) but without any visual cue this has happened (bad!) which can lead to real world/real human misunderstandings and frustration ;)

Also, it's somewhat inconsistent in terms of experience with the fact that when messages have failed to send, riot offers to resend them, showing it's clearly aware of intermittent connectivity issues :)

@ara4n ara4n added T-Defect S-Major Severely degrades major functionality or product features, with no satisfactory workaround P2 A-Room-View S-Minor Impairs non-critical functionality or suitable workarounds exist and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Feb 9, 2017
@richvdh
Copy link
Member

richvdh commented Apr 18, 2017

also related to #1900?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

4 participants