Skip to content

Conversation

dab246
Copy link
Member

@dab246 dab246 commented Jul 30, 2025

Issue

#3864

Resolved

Screen.Recording.2025-07-30.at.15.16.23.mov

@dab246 dab246 requested review from hoangdat and tddang-linagora and removed request for hoangdat July 30, 2025 08:20
Copy link

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

@hoangdat
Copy link
Member

hoangdat commented Aug 1, 2025

  • what happen if I combine multiple email like this
john.doe@example.com, jane.smith@example.com, alex.wilson@example.com, "ng van a" <ttnn@gmail.com>

@dab246
Copy link
Member Author

dab246 commented Aug 1, 2025

  • what happen if I combine multiple email like this
john.doe@example.com, jane.smith@example.com, alex.wilson@example.com, "ng van a" <ttnn@gmail.com>

It will work in the old thread separating by character between emails

@hoangdat
Copy link
Member

hoangdat commented Aug 1, 2025

It will work in the old thread separating by character between emails

But it failed with new format

image

@hoangdat
Copy link
Member

hoangdat commented Aug 1, 2025

  • emails be combined in one tag if I paste like this format
image
"ng van a" <ttnn@gmail.com> "ng van b" <ttnn1@gmail.com> "ng van c" <ttnn2@gmail.com> "ng van d" <ttnn3@gmail.com>

@dab246
Copy link
Member Author

dab246 commented Aug 1, 2025

It will work in the old thread separating by character between emails

But it failed with new format

image

It will work in the old thread separating by character between emails

But it failed with new format

Because It separates by white space too.

@dab246
Copy link
Member Author

dab246 commented Aug 1, 2025

  • emails be combined in one tag if I paste like this format
"ng van a" <ttnn@gmail.com> "ng van b" <ttnn1@gmail.com> "ng van c" <ttnn2@gmail.com> "ng van d" <ttnn3@gmail.com>

Expection? Currently we don't support multiple "ng van a" ttnn@gmail.com

@hoangdat
Copy link
Member

hoangdat commented Aug 1, 2025

Currently we don't support multiple "ng van a" ttnn@gmail.com

if we don't support, should not combine it into a tag

dab246 added 3 commits August 3, 2025 23:19
Signed-off-by: dab246 <tdvu@linagora.com>
Signed-off-by: dab246 <tdvu@linagora.com>
@dab246 dab246 force-pushed the enhancement/tf-3864-pasting-address-with-display-name branch from c5e9dd9 to 7f19398 Compare August 4, 2025 02:45
@dab246
Copy link
Member Author

dab246 commented Aug 4, 2025

  • Support pasting multiple address with multiple format mixed
demo.mov

@dab246 dab246 requested a review from tddang-linagora August 4, 2025 02:47
Signed-off-by: dab246 <tdvu@linagora.com>
@dab246 dab246 requested a review from tddang-linagora August 5, 2025 04:36
@hoangdat hoangdat merged commit 6e86946 into master Aug 19, 2025
23 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.

3 participants