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-3278 Mobile handle corner cases of TWP redirection #3283

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

dab246
Copy link
Member

@dab246 dab246 commented Nov 19, 2024

Issue

#3278

Deep Links Config

  • Uri format: twakemail.mobile://openApp?access_token=abc&refresh_token=acb123&id_token=abc&expires_in=36000&[email protected]

Resolved

user signed in SignUp page
user want to open tmail app from SignUp page
but tmail app was not installed 
open tmail store
usecase1.webm
user signed in SignUp page
user want to open tmail app from SignUp page
but tmail app was installed but not signed in
sign in app automatically
usecase2.webm
usecase2.mp4
user signed in SignUp page
user want to open tmail app from SignUp page
and tmail app was installed but signed in other account
notice user about it
  • Android App terminated:
usecase3.webm
  • Android App foreground:
usecase5.webm
  • Android App with open composer:
usecase6.webm
  • iOS
usecase3.mp4
user signed in SignUp page
user want to open tmail app from SignUp page
and tmail app was installed but signed in with the same account
open tmail app as normal
usecase4.webm
usecase4.mp4

Copy link

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

@dab246
Copy link
Member Author

dab246 commented Nov 21, 2024

  • Work well on stg.lin-saas.com
demo-1.webm
demo-2.online-video-cutter.com.mp4

@dab246
Copy link
Member Author

dab246 commented Nov 21, 2024

Propose

We are now using a different authentication method than the old flow before. That is, parsUri itself from the redirect url returned from TW. So we need some parameters that are required to be flexible in developing many TWs.

  • access_token: MUST HAVE
  • refresh_token, id_token, expires_in: Optional
  • username: MUST HAVE
  • registration_url: MUST HAVE
  • jmap_url: MUST HAVE

E.g:

twakemail.mobile://openApp?access_token=abc&refresh_token=acb123&id_token=abc&expires_in=36000&[email protected]&registration_url=https://sign-up.example.com&jmap_url=https://jmap.example.com

@hoangdat

@dab246
Copy link
Member Author

dab246 commented Nov 23, 2024

Propose

We are now using a different authentication method than the old flow before. That is, parsUri itself from the redirect url returned from TW. So we need some parameters that are required to be flexible in developing many TWs.

  • access_token: MUST HAVE
  • refresh_token, id_token, expires_in: Optional
  • username: MUST HAVE
  • registration_url: MUST HAVE
  • jmap_url: MUST HAVE

E.g:

twakemail.mobile://openApp?access_token=abc&refresh_token=acb123&id_token=abc&expires_in=36000&[email protected]&registration_url=https://sign-up.example.com&jmap_url=https://jmap.example.com

@hoangdat

Has been processed

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.

2 participants