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

fix: "jump down" button incorrect badge count #4593

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

WofWca
Copy link
Collaborator

@WofWca WofWca commented Feb 2, 2025

...it show the value from another chat.

This can happen if the chat has been switched, but no relevant
events occurred after the chat has been switched.

Closes #3978.

The second commit should not affect behavior, unless the core docs are wrong.

I only tested that the reproduction steps from #3978 (comment) are no longer reproducible. I did not test too deeply.

@WofWca WofWca force-pushed the wofwca/fix-jump-down-counter branch from f299816 to 51c978e Compare February 2, 2025 16:51
...it show the value from another chat.

This can happen if the chat has been switched, but no relevant
events occurred after the chat has been switched.

Closes #3978.
`MsgRead` only applied to outgoing messages, so it doesn't affect
the value returned from `BackendRemote.rpc.getFreshMsgCnt()`.
@WofWca WofWca force-pushed the wofwca/fix-jump-down-counter branch from 51c978e to 6c1f9ef Compare February 2, 2025 16:53
Copy link
Member

@nicodh nicodh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected.

One comment why the MsgRead event is removed

@WofWca WofWca merged commit ff5d544 into main Feb 6, 2025
12 checks passed
@WofWca WofWca deleted the wofwca/fix-jump-down-counter branch February 6, 2025 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

go-down-arrow shows wrong unread badge counter sometimes
2 participants