Skip to content

Email Client - #199

Draft
domai-tb wants to merge 36 commits into
masterfrom
Email-Client
Draft

Email Client#199
domai-tb wants to merge 36 commits into
masterfrom
Email-Client

Conversation

@domai-tb

@domai-tb domai-tb commented May 19, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

E-Mail Client

This update introduces an integrated E-Mail client for RUB mails.

Features

  • E-Mail Client UI: Write, Edit, Delete, Show etc.
  • IMAP-Anbindung der E-Mails und Directories / Folder

Code Quality / Internals

  • Unit-Testing (ideally)

@domai-tb

domai-tb commented Jun 30, 2025 •
edited by Aldalati
Loading

Copy link
Copy Markdown
Member Author

Hey @H1b4x thank your for your work! 🙏🏼

Some task that need to be done before we can release the email client in production:

  • Different email folders should not "stack". The backwards button should always lead to the "More" page instead of the inbox. Currently, folders like send, trash etc. are "stacked" on top of the view.
  • Load user defined folders from server. Currently it just loads the inbox, spam, send, archives, drafts and trash.
  • The settings button should either removed entirely or some settings should be implemented. The settings should be united with the general app settings page.
  • A background service for cyclic fetching up-to-date emails and displaying a notification should be implemented.
  • Drafts aren't stored properly on server side. See [1]
  • Handle @media CSS within HTML body. See Flutter HTML Issue 1060. Should be implemented within the SytledHtml widget.
  • Stabilize widget tree and avoid dynamic calls (over async gaps) to handle Unhandled Exception: Looking up a deactivated widget's ancestor is unsafe. At this point the state of the widget's element tree is no longer stable.
  • Implement and handle attachments properly.

[1]
Screenshot_2025-06-30_13_26_51

@domai-tb

domai-tb commented Aug 3, 2025

Copy link
Copy Markdown
Member Author

We need to manually handle the dependency enough_email due the outdated package on pub.dev. See this issue. The error message with an up-to-date SDK is the following:

❯ flutter pub add enough_mail
The current Dart SDK version is 3.8.1.

Because enough_mail <1.0.0 doesn't support null safety and enough_mail >=1.0.0 <2.1.2 depends on intl ^0.17.0, enough_mail <2.1.2 requires intl ^0.17.0.
And because enough_mail >=2.1.2 <2.1.6 depends on intl ^0.18.0 and enough_mail >=2.1.6 depends on intl ^0.19.0, every version of enough_mail requires intl ^0.17.0 or ^0.18.0 or ^0.19.0.
So, because campus_app depends on both intl ^0.20.2 and enough_mail any, version solving failed.

The lower bound of "sdk: '>=2.7.0 <3.0.0'" must be 2.12.0 or higher to enable null safety.
For details, see https://dart.dev/null-safety
Failed to update packages.

As the package maintainer doesn't seem to provide a update in the near future, we should fork the repository like we did with the video player plugin too.

@domai-tb domai-tb mentioned this pull request Aug 27, 2025
Open
5 tasks
@domai-tb domai-tb added this to the v2.5 milestone Oct 7, 2025
domai-tb and others added 6 commits October 21, 2025 12:40
@domai-tb
Merge remote-tracking branch 'origin/HEAD' into Email-Client
53c9c08
@Aldalati
dynamic IMAP folder loading , fixed folder stacking with pushReplacem…
3c81f7c 
…ent, and implemented backgground fetch with authentication and UID based notification
@Nicole-Dhm
fix: updated sentry_flutter version
4088123 
- the branch previously did not run due to an unsupported Kotlin version error, with the sentry_flutter update this branch builds again
@Nicole-Dhm
Merge remote-tracking branch 'origin/master' into Email-Client
958b35f 
- resolved merge conflicts
- changed login screen implementation to be compatible with master branch (email Client now has its own page until a central login screen replaces all individual login screens)
- reset ticket login screen implementation to master version
@Nicole-Dhm
fix: email login progress indicator
670257b 
- created an authentication page that handles the visual loading while authenticating and fetching emails
- seperated UI loading concern from login screen
- added back button to loading page
- fixed navigation after logout (return to more page after logout)
- loading now works correctly after inital login
@Nicole-Dhm
chore: improved email fetching and email bodies
0c4f629 
- we now do lazy fetching for the emails -> we fetch the headers only and then the bodies when an email is opened
- the email bodies have been improved (MIME Tree structure), most if not all email bodies are now displayed (primarily in plain text -> displaying as many HTML bodies successfully as possible is still a TODO)
- added fetching emails by their UID into services and repo layers
- Flags are now fetched properly
@Nicole-Dhm Nicole-Dhm assigned Nicole-Dhm and unassigned H1b4x, Nicole-Dhm and Aldalati May 1, 2026
Nicole-Dhm added 10 commits May 4, 2026 17:56
@Nicole-Dhm
feat: handle IMAP connection timeout
879b519 
- we now check before every IMAP operation if our IMAP connection is alive, if it isn't we try to reset and reconnect
- the previous check wasn't catching the SocketException, this check is now replaced by the wrapper function managing the connection status
@Nicole-Dhm
fix: delayed IMAP reconnect + HTML bodies improvement
5b6a8fb 
- resolved the delay in the IMAP reconnect with a function timeout to force a reconnect when we can't detect the connection loss with the exceptions and errors
- broadend erros being caught in the reconnect cycle
- added a keep alive that pings the IMAP client with a NOOP every 5 minutes, this will also drastically reduce the amount of times we need to iniate a whole reconnect
- switched to InAppWebView to display html email bodies (still TODO: refactor layout when an email displays HTML, because SingleChildScrollView and InAppWebView don't mesh well)
- exported loading of the email bodies to the email view itself instead of being called from email pages (for this email view was converted to a stateful widget)
- added loading spinner to email bodies loading
@Nicole-Dhm
chore: refactored email view layout
43ce0a6 
- to accomadate the InAppWebView better the layout now has different cases for plaintext and html bodies, plaintext has the same scrolling behaviour as before, scrolling the header. HTML bodies now have a fixed header so the scrolling in the web view works much smoother and doesn't figth with an outer scroll behaviour
- since we block images in HTML bodies per default (for privacy), we check if a HTML body email has images and give the user a load button that lets them load in the images
- web views now no longer handle their own height dynamically but expand to the size they need right away, which is much more robust
- extracted widgets into build functions to improve overview of the widget tree and avoid redundant code
@Nicole-Dhm
fix: duplicate drafts, sender email and display name
73a6279 
- when saving a draft, we now no longer have mulitple duplicate drafts and only save one draft, "replaacing" the draft each time
- since the rub dovecot server does not support the IMAP replace operation we save a new draft with a combination of delete the existing one and appending the new one
- we now ask the user for their email address on init, since we do not have direct access to it. It is saved in the secure storage
- we check the email address for the @ format combined with the RUB email .de ending
- from the email we automatically generate a suggested Display Name, which the user can choose to overwrite. It is also saved in the secure storage
@Nicole-Dhm
fix: draft bodies not showing
a1e7660 
- we now save the draft bodies correctly in the MIME format
- draft bodies now display correctly
- drafts only save on PopScope when the content has truly changed
- we're keeping track of the drafts with the help of their Email headers now, so as to not save mutliple versions
- opening the Drafts drawer is now correctly routed to the drafts page
- drafts are now editable
@Nicole-Dhm
fix: move and delete operations
6be5240 
- the move operation is now permanent server side
- the move fix also implicitly fixed part of the delete operation that moves emails to the trash folder
- the delete operation is now permanent and deleting the right emails with the combination of uid's and mailboxnames
- added the display name to sent emails
- changed snackbar notificaitons in email view to global bubbles messages
@Nicole-Dhm
feat: added Email pull up sheet + fix: email service move
703c573 
- added a more sheet with additional actions for an Email (currently: marking read status, Move Email to Folder, Spam marking)
- fixed the service move, now any move from any folder to another sticks around
- now keeping track of the new UID assigned after a move by the IMAP server
- refactored the email service move function
- added convinience wrappers to the move function (move multiple emails and move via EmailFolder or MailboxName)
@Nicole-Dhm
feat: added server side email search
784ae3c 
- we now search for emails server side (prev: filter on locally stored emails), now using the right IMAP function
- restructured fetching of emails by their UID slightly
- to prevent enormous loading times when searching for emails we dynamically load the matched emails by first retrieving all the UIDs of matching emails, then batch loading the emails in as needed
- the next batch triggers when the user scrolls down far enough
- loading speed could still be improved (might be beneficial to change from load as typing to confirm after typing)
- UI update to searching: while searching now portraying a loading spinner and informing the User when no emails are found
- when in search bar background is now blanked out
- added rebuild function to trigger rebuilds since search is now actually doing the searching
@Nicole-Dhm
misc: clean up unused files, improve webview, optimize search
0728f74 
- removed mutliple unused files: archives (not a native rub email folder), email settings (empty file), sent file (is managed by dynamic folder building)
- changed search while typing Search to a search after confirm; less risk of searching unnecessary unfinished search terms and showing outdated emails
- urls are now handed off to the OS from the webview, it does not handle opening them
- now resolving inline attached images in addition to the network images, which allows us to show more images in the webview
- optimized search: previously search fell into an unnecessary reconnect since it takes a while to search through a unindexed mailbox; now we have a custom timeout param so searches can have a 90 sec timeout window
- we now try to cancel a previous outdated search, if a new search is started
@Nicole-Dhm
chore: add bottom sheet to compose screen
8917be6 
- added bottom sheet to the compse screen with a modified sheet, having a different version for drafts
- fixed bug in fetchfullemail (folder was wrongly overwritten)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@H1b4x H1b4x

@Nicole-Dhm Nicole-Dhm

@Aldalati Aldalati

Labels

feature:email

Projects

Campus App by AStA Roadmap
Status: Paused

Milestone

v2.5

Development

Successfully merging this pull request may close these issues.

4 participants

@domai-tb @H1b4x @Nicole-Dhm @Aldalati