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

Mark as read not sending the correct command to mark as read #27777

Closed
niebloomj opened this issue Jul 18, 2024 · 7 comments
Closed

Mark as read not sending the correct command to mark as read #27777

niebloomj opened this issue Jul 18, 2024 · 7 comments
Labels
A-Read-Marker Green line showing how far _you_ have read O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Cannot-Reproduce

Comments

@niebloomj
Copy link

niebloomj commented Jul 18, 2024

Steps to reproduce

  1. Where are you starting? What can you see?

I see a chat that is marked as unread (without a number next to it)
CleanShot 2024-07-18 at 08 52 54@2x

  1. What do you click?
    I click this button
    CleanShot 2024-07-18 at 08 53 14@2x

  2. I see that the chat is NOT marked as read

  3. I type /devtools to see a developer popup

  4. I see that the state is marked as unread here
    CleanShot 2024-07-18 at 08 55 14@2x

  5. I edit the message and update the JSON to be an empty dict.

  6. I click send

  7. I see that the chat is now marked as read properly. I assume this unread state is being considered for the indicator but not one of the states that are updated by clicking mark as read.

Outcome

What did you expect?

I expected the chat to be marked as read

What happened instead?

It remained unread.

Operating system

macOS

Application version

Version 2024071801 (2024071801)

How did you install the app?

Github Download

Homeserver

Dendrite 0.13.7

Will you send logs?

Yes

@dosubot dosubot bot added A-Read-Marker Green line showing how far _you_ have read O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist labels Jul 18, 2024
@dbkr
Copy link
Member

dbkr commented Jul 22, 2024

In step 3, did you mean 'read' or 'unread'?

This could be a bug with stable / unstable prefix support for mark as unread though, since the devtools show it's been set with the stable prefix (although we don't know with what client).

@dbkr dbkr added O-Occasional Affects or can be seen by some users regularly or most users rarely and removed O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Jul 22, 2024
@niebloomj
Copy link
Author

Step 3 wording is correct. After I click the button, the state does not change and the chat is not being marked as read.

@niebloomj
Copy link
Author

I noticed the change from frequent to occasional. If I understand that label correctly, this is in fact happening incredibly frequently. Anytime that state is present for a specific chat, mark as read does not work.

@t3chguy
Copy link
Member

t3chguy commented Jul 22, 2024

image

@niebloomj please read the label description. Given this is only a single user's report of such, it can't be Frequent.

@niebloomj
Copy link
Author

Gotcha thanks! I didn't see that on mobile.

@niebloomj
Copy link
Author

Issue still persists on latest element nightly and denrite main branch

@dbkr dbkr changed the title Make as read not sending the correct command to mark as read Mark as read not sending the correct command to mark as read Oct 29, 2024
@t3chguy
Copy link
Member

t3chguy commented Nov 25, 2024

Fixed by #28478 prior to which we only sent the unstable but would prefer the stable which could cause such a "stuck" unread

@t3chguy t3chguy closed this as completed Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Read-Marker Green line showing how far _you_ have read O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Cannot-Reproduce
Projects
None yet
Development

No branches or pull requests

3 participants