Skip to content

Conversation

@dab246
Copy link
Member

@dab246 dab246 commented Nov 10, 2023

Issue

#2271

Dependent

Resolved

1. Found Jmap Url

Simulator.Screen.Recording.-.iPhone.14.Pro.Max.-.2023-11-13.at.15.26.50.mp4

2. Not found Jmap Url

Simulator.Screen.Recording.-.iPhone.14.Pro.Max.-.2023-11-13.at.15.27.26.mp4

3. With Oidc

Simulator.Screen.Recording.-.iPhone.14.Pro.Max.-.2023-11-13.at.15.28.18.mp4

@github-actions
Copy link

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

@chibenwa
Copy link
Member

chibenwa commented Nov 10, 2023

Don't ask for email address twice when the user need to input JMAP URL IMO (on video 0:30).

@chibenwa
Copy link
Member

Can you do a demo video for [email protected] ?

@dab246 dab246 force-pushed the improvement/tf-2271-jmap-endpoint-auto-discovery branch from a00c6f1 to 9744af8 Compare November 13, 2023 08:15
@dab246 dab246 force-pushed the improvement/tf-2271-jmap-endpoint-auto-discovery branch from 9744af8 to 8d560d1 Compare November 13, 2023 08:22
@hoangdat
Copy link
Member

@chibenwa can we change the SRV record to the jmap with OIDC?

@dab246
Copy link
Member Author

dab246 commented Nov 13, 2023

Can you do a demo video for [email protected] ?

@chibenwa With the email [email protected] we will not be able to find the SRV record. Below demo

Simulator.Screen.Recording.-.iPhone.14.Pro.Max.-.2023-11-13.at.16.14.03.mp4

@dab246 dab246 force-pushed the improvement/tf-2271-jmap-endpoint-auto-discovery branch from 8d560d1 to b6708ad Compare November 17, 2023 13:03
@dab246
Copy link
Member Author

dab246 commented Nov 17, 2023

@hoangdat Please review it.

@dab246
Copy link
Member Author

dab246 commented Nov 20, 2023

@hoangdat Please review again

@dab246 dab246 merged commit c581798 into master Nov 21, 2023
@hoangdat hoangdat deleted the improvement/tf-2271-jmap-endpoint-auto-discovery branch February 20, 2024 04:39
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