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

TF-3396 Fix mobile app can not get the latest email when open app from terminated #3400

Conversation

dab246
Copy link
Member

@dab246 dab246 commented Jan 6, 2025

Issue

#3396

Root cause

  • Since we skipped get change when switching between mailboxes, we didn't get the latest changes when the app was in terminated state.

Solution

  • To solve this problem, we perform get change emails the first time we open the app. This solution is useful when we disable Websocket or FCM and always receive the latest changes when opening the app.

Resolved

  • Mobile:
demo.online-video-cutter.com.mp4
  • Web:
demo.mov

Copy link

github-actions bot commented Jan 6, 2025

This PR has been deployed to https://linagora.github.io/tmail-flutter/3400.

@dab246 dab246 force-pushed the bugfix/tf-3396-mobile-app-can-not-get-the-latest-email-when-open-app-from-terminated branch from fac5358 to f30749a Compare January 6, 2025 08:25
@chibenwa
Copy link
Member

chibenwa commented Jan 6, 2025

@hoangdat any chance to get this timely?

@hoangdat
Copy link
Member

hoangdat commented Jan 6, 2025

@hoangdat any chance to get this timely?

ASAP, but not have much value for web app.

@dab246 dab246 force-pushed the bugfix/tf-3396-mobile-app-can-not-get-the-latest-email-when-open-app-from-terminated branch from f30749a to a60cde5 Compare January 7, 2025 01:49
@dab246
Copy link
Member Author

dab246 commented Jan 7, 2025

ASAP, but not have much value for web app.

I have updated the most effective solution, it works well on both mobile and web.

@hoangdat hoangdat merged commit 9ceb56c into maintenance-v0.14.2 Jan 7, 2025
22 checks passed
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.

4 participants