Missing thread event causes stuck notification

[richvdh]

Steps to reproduce

This won't go away no matter how much I open the room, scroll around, open the threads panel, ditto:

Outcome

Operating system

Ubuntu 22.04

Application version

Element Nightly version: 2022121801 Olm version: 3.2.12

How did you install the app?

No response

Homeserver

No response

Will you send logs?

Yes, https://github.com/matrix-org/element-web-rageshakes/issues/18581

Discussed at https://matrix.to/#/!tCaHWuHHvOHjWyqapP:matrix.org/$NJt7bWWvJJkfg-q40xnPU4D17MOnIlR6qZcQd1fBkIM?via=matrix.org&via=element.io&via=riot.ovh

[richvdh]

It turns out I had missed an active thread; opening it reduced the unread count from 3 to 2.

Comparing with @MadLittleMods' view of the same thread, it appears I am missing the two most recent messages in the thread.

[Z-Brox]

Can confirm the problem - my Client (Element: 1.11.16, Olm: 3.2.12, Debian 11) shows the notification marker stuck at "1" for my favourite room. Deleted the cache and restartet the client, but that didn't help.
My other client (1.5.11, Android) works without problems with the same room showing no unread notifications.

[weeman1337]

Comparing with @MadLittleMods' view of the same thread, it appears I am missing the two most recent messages in the thread.

That sounds like another problem @richvdh . Can you open another issue for it?

[richvdh]

Comparing with @MadLittleMods' view of the same thread, it appears I am missing the two most recent messages in the thread.

That sounds like another problem @richvdh . Can you open another issue for it?

I mean I can, but isn't it likely to be the cause of my stuck notification marker? I can retitle this one instead?

[Z-Brox]

Just received an update (to version 1.11.17), the bug is gone. Thank you :-) .

[germain-gg]

I have a feeling that matrix-org/matrix-js-sdk#3002 will have a positive impact on this.

The stuck notifications are likely caused by the missing events in the thread, so it can't send a read receipt.
I managed to reproduce this issue once and saw that the pagination token was missing.

Have not yet figured out what caused it and how to reproduce reliably

[weeman1337]

matrix-org/matrix-js-sdk#3002 got merged. Can you test again on develop @richvdh ?

[richvdh]

I can't reproduce it any more with the previous version, so 🤷‍♂️

[turt2live]

Closing as presumed fixed. If this is untrue, please open a new issue with fresh reproduction steps for the QA team to review.