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.

Sign up for GitHub

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

Jump to bottom

TF-3157 Implement web socket push #3168

Merged

hoangdat merged 5 commits into master from feature/TF-3157-implement-web-socket-push

Nov 6, 2024

Contributor

tddang-linagora commented Sep 23, 2024 •

edited

Loading

Issue

Implement websocket to Twake Mail #3157

Demo

tddang-linagora requested a review from hoangdat

September 23, 2024 06:39

tddang-linagora temporarily deployed to PR-3168

September 23, 2024 06:39

— with

GitHub Actions Inactive

tddang-linagora requested a review from dab246

September 23, 2024 06:39

tddang-linagora self-assigned this

github-actions bot commented Sep 23, 2024

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

tddang-linagora temporarily deployed to PR-3168

September 23, 2024 09:55

— with

GitHub Actions Inactive

tddang-linagora marked this pull request as ready for review

September 23, 2024 09:55

dab246 reviewed

View reviewed changes

lib/features/base/action/ui_action.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource/web_socket_datasource.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource_impl/remote_web_socket_datasource_impl.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource_impl/remote_web_socket_datasource_impl.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource_impl/remote_web_socket_datasource_impl.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource_impl/remote_web_socket_datasource_impl.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource_impl/remote_web_socket_datasource_impl.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/domain/state/web_socket_push_state.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/domain/usecases/connect_web_socket_interactor.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/presentation/controller/push_base_controller.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/presentation/controller/push_base_controller.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/presentation/controller/push_base_controller.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/presentation/controller/push_base_controller.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/presentation/controller/web_socket_controller.dart Outdated Show resolved Hide resolved

Member

dab246 commented Sep 24, 2024 •

edited by tddang-linagora

Loading

Verify web socket when token expired
Verify web socket on background mode

tddang-linagora temporarily deployed to PR-3168

September 25, 2024 04:19

— with

GitHub Actions Inactive

tddang-linagora changed the title ~~[WIP] TF-3157 Implement web socket push~~ TF-3157 Implement web socket push

tddang-linagora requested a review from dab246

September 25, 2024 04:20

hoangdat changed the title ~~TF-3157 Implement web socket push~~ [WIP] TF-3157 Implement web socket push

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource_impl/remote_web_socket_datasource_impl.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource_impl/remote_web_socket_datasource_impl.dart Outdated Show resolved Hide resolved

tddang-linagora temporarily deployed to PR-3168

September 30, 2024 03:30

— with

GitHub Actions Inactive

dab246 reviewed

View reviewed changes

lib/features/push_notification/data/datasource_impl/remote_web_socket_datasource_impl.dart Outdated Show resolved Hide resolved

dab246 reviewed

View reviewed changes

Member

dab246 left a comment •

edited by tddang-linagora

Loading

Please remove firebase-messaging-sw.js file in web folder when we already used weg socket

tddang-linagora temporarily deployed to PR-3168

November 5, 2024 03:12

— with

GitHub Actions Inactive

dab246 reviewed

View reviewed changes

Member

dab246 left a comment •

edited by tddang-linagora

Loading

Remove code fcm on web

dab246 reviewed

View reviewed changes

core/lib/data/constants/constant.dart Outdated Show resolved Hide resolved

tddang-linagora temporarily deployed to PR-3168

November 5, 2024 03:31

— with

GitHub Actions Inactive

tddang-linagora requested a review from dab246

November 5, 2024 03:31

Member

dab246 commented Nov 5, 2024 •

edited by tddang-linagora

Loading

Sometime not work on chrome: Go to setting -> send email -> back to dashboard screen -> not update

Screen.Recording.2024-11-05.at.10.36.31.mov

hoangdat had a problem deploying to prod

November 5, 2024 04:33

— with

GitHub Actions Error

hoangdat temporarily deployed to prod

November 5, 2024 04:33

— with

GitHub Actions Inactive

hoangdat had a problem deploying to prod

November 5, 2024 04:33

— with

GitHub Actions Error

dab246 reviewed

View reviewed changes

lib/features/push_notification/presentation/services/fcm_receiver.dart Outdated Show resolved Hide resolved

tddang-linagora temporarily deployed to PR-3168

November 5, 2024 07:59

— with

GitHub Actions Inactive

tddang-linagora requested a review from dab246

November 5, 2024 08:00

dab246 reviewed

View reviewed changes

lib/features/push_notification/presentation/controller/web_socket_controller.dart Outdated Show resolved Hide resolved

tddang-linagora temporarily deployed to PR-3168

November 5, 2024 08:20

— with

GitHub Actions Inactive

dab246 reviewed

View reviewed changes

Member

dab246 left a comment •

edited by tddang-linagora

Loading

Please rebase it

tddang-linagora added 4 commits

November 5, 2024 15:57


          TF-3157 Update jmap-dart-client dependency

356cc98


          TF-3157 Implement web socket push

39728b6


          TF-3157 Update web socket with background service worker

c5f49b9

TF-3157 Stub BroadcastChannel for mobile build


          TF-3157 Remove unnecessary background service worker

75e8ca9

tddang-linagora force-pushed the feature/TF-3157-implement-web-socket-push branch from 008823e to 75e8ca9 Compare

November 5, 2024 08:57

tddang-linagora temporarily deployed to PR-3168

November 5, 2024 08:57

— with

GitHub Actions Inactive

hoangdat reviewed

View reviewed changes

docs/adr/0053-web-socket-data-synchronization.md Outdated Show resolved Hide resolved


          fixup! TF-3157 Implement web socket push

379a213

tddang-linagora deployed to PR-3168

November 6, 2024 02:09

— with

GitHub Actions View deployment

tddang-linagora requested review from hoangdat and dab246

November 6, 2024 02:10

dab246 approved these changes

View reviewed changes

Member

dab246 left a comment

lgtm

hoangdat approved these changes

View reviewed changes

hoangdat merged commit 670d1aa into master

22 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet