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

Websocket notif messes up the order of my INBOX #3453

Open
chibenwa opened this issue Jan 26, 2025 · 3 comments
Open

Websocket notif messes up the order of my INBOX #3453

chibenwa opened this issue Jan 26, 2025 · 3 comments
Assignees
Labels
bug Something isn't working Critical

Comments

@chibenwa
Copy link
Member

Description

On Chromium when I receive a websocket notification, Twake mail does a rogue Email/query + Email/set that completly mess up the sort order of my mails (ordered after email/get and not email/query)

notif.mp4

Version: 0.14.11

Problem:

  • it do NOT happen on the exact same browser with a different account.
  • it still happen after f5 + disconnect reconnect
  • it do not happen with the same account and a different browser

Expected result

  • Do not do this uneeded Email/query + Email/get
  • (never order according to Email/get but always according to Email/query)

Current behavior

Websocket notifs messes up the content of my INBOX

Requests

  1. Email/set doing the action

Image

(Wait 2 seconds)

Image

  1. Normal state resync

Image

  1. Immediatly followed by the rogue resync...
@chibenwa chibenwa added bug Something isn't working Critical labels Jan 26, 2025
@chibenwa
Copy link
Member Author

I have this too on mobile !!!

V 0.14.12

@dab246 dab246 self-assigned this Feb 3, 2025
@dab246
Copy link
Member

dab246 commented Feb 3, 2025

IMO, This error is related to the sort function after executing get all emails. It's similar to #3298. It was fixed at PR #3299 and merged on master branch

@dab246
Copy link
Member

dab246 commented Feb 3, 2025

@hoangdat We should cherry pick these commit from master to maintenance-v0.14.2 branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Critical
Projects
None yet
Development

No branches or pull requests

2 participants