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.

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-2370 Cannot send email on Stalwart #2398

Open
wants to merge 1,048 commits into
base: master
Choose a base branch
from
Open

TF-2370 Cannot send email on Stalwart #2398

wants to merge 1,048 commits into from
+67,664 −29,305

Conversation

dab246
Copy link
Member

@dab246 dab246 commented Dec 10, 2023 •
edited
Loading

Issue

#2370

Root cause

  • Mailbox/changes on Stalwart does not work properly.
  • Folder Outbox has already been created, so it cannot be created anymore.

Solution

  • We will separate the Outbox handling function in SendEmailInteractor. Do not handle nested method in jmap request anymore.

Demo

  • Server StartWalt:
demo_stalwart.mov
  • Server Linagora:
Screen.Recording.2023-12-11.at.12.49.05.mov

dab246 and others added 30 commits October 4, 2023 09:15
@dab246
TF-2172 Display DropZone view when drag active
13a88cc
@dab246
TF-2172 Handle add attachment from drop zone
a9c2b8d
@dab246
TF-2172 Disable focus of HtmlContentView when dragging attachment
2fdbc6a
@dab246
TF-2202 Use InternetConnectionChecker to check internet connection …
451ebbd 
…on mobile device
@dab246
TF-2202 Store SendingEmail to queue when get error `NoInternetConne…
2e0b5ca 
…ction` from server
@hieutbui @dab246
TF-2199 show suggestion for unknown email when suggestion list is empty
3aad768
@hieutbui @dab246
TF-2199 show suggestion for unknown email when suggestion list is not…
89a86c7 
… empty
@dab246
TF-2189 Keep composer available before click on proceed
a818b91
@dab246
TF-2189 Handle click cancel on dialog
c0a7af2
@dab246
TF-2189 Avoid compile runtime InternetConnectionChecker on web
7052f3e
@dab246
TF-2182 Fix no blue bar for calendar events
7ea91d5
@dab246
TF-2182 Add video conferences for google event
9ab9a9b
@dab246
TF-2182 Add event actions for google event
3ca22fd
@dab246
TF-2182 Enable hyperlink for event description
5b44a00
@dab246 @hoangdat
TF-2188 Add Canceled sending state
fb802e5
@dab246 @hoangdat
TF-2188 Allow editable waiting/running/canceled status
04af133
@dab246 @hoangdat
TF-2188 Disable sending queue on iOS
cec4ccb
@dab246 @hoangdat
TF-2188 Remove radius and background color bottom bar sending queue
03d03dd
@dab246 @hoangdat
TF-2190 Add top padding for confirm dialog
47d1c2e
@dab246 @hoangdat
TF-2187 Handle click system back button to back to Inbox
95dfd0e
@dab246 @hoangdat
TF-2176 Update app grid icons default
2104bad
@hieutbui @dab246
TF-2194 Add newSubfolder to MailboxActions
7906766
@hieutbui @dab246
TF-2094 add newSubFolder action to list action for default and person…
71eff81 
…al mailbox
@hieutbui @dab246
TF-2194 Update mailbox creator for pre-selected mailbox parent
c8b2dc1
@hieutbui @dab246
TF-2194 Add case newSubfolder to handleMailboxAction
6f41eb6
@hieutbui @dab246
TF-2194 Update create new mailbox without pre-selected parent mailbox
84ca809
@hieutbui @dab246
TF-2194 Add case subFolder to search mailbox
ee0b9b3
@dab246
Set default mimeType is application/octet-stream
14b0ff9
@dab246
Fix some file with mimeType is empty cannot drag n drop attachment
ad223e1
@dieptran88 @dab246
Translated using Weblate (Vietnamese)
7032784 
Currently translated at 100.0% (502 of 502 strings)

Translation: Linagora/TeamMail
Translate-URL: https://hosted.weblate.org/projects/linagora/teammail/vi/
@dab246 dab246 mentioned this pull request Dec 10, 2023
Open
@github-actions GitHub Actions
Copy link

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

@dab246 dab246 force-pushed the bugfix/tf-2370-cannot-send-email-on-stalwart branch from 38136bf to 019c7a1 Compare December 11, 2023 05:56
@dab246 dab246 force-pushed the bugfix/tf-2370-cannot-send-email-on-stalwart branch from 019c7a1 to cd75757 Compare December 11, 2023 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hoangdat hoangdat Awaiting requested review from hoangdat

@hieutbui hieutbui Awaiting requested review from hieutbui

@sherlockvn sherlockvn Awaiting requested review from sherlockvn

@nqhhdev nqhhdev Awaiting requested review from nqhhdev

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
stalwart
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@dab246 @hieutbui @dieptran88 @Atalanttore @tk-nguyen @chibenwa @hoangdat @patHyatt @davopo