Skip to content

[_]: feature/get all emails from all mailboxes#13

Merged
xabg2 merged 3 commits intomasterfrom
feature/get-all-emails-from-all-mailboxes
Apr 7, 2026
Merged

[_]: feature/get all emails from all mailboxes#13
xabg2 merged 3 commits intomasterfrom
feature/get-all-emails-from-all-mailboxes

Conversation

@xabg2
Copy link
Copy Markdown
Contributor

@xabg2 xabg2 commented Apr 6, 2026

Getting all emails for a given user from all mailboxes (used to sync the local DB - indexed DB with the user emails).

@xabg2 xabg2 requested a review from jzunigax2 April 6, 2026 09:36
@xabg2 xabg2 self-assigned this Apr 6, 2026
@xabg2 xabg2 added the enhancement New feature or request label Apr 6, 2026
@jzunigax2 jzunigax2 force-pushed the feature/get-all-emails-from-all-mailboxes branch from 1c74a7c to 94e0cbb Compare April 6, 2026 15:13
- Removed DomainRepository export from AccountModule.
- Updated AccountService to include a new method for listing active domains.
- Refactored EmailController to use user email instead of a stubbed user for various email operations.
- Enhanced JmapMailProvider to utilize a new queryEmails method for improved email fetching logic.
- Added unit tests for new email retrieval functionality and adjusted existing tests for consistency.
- Renamed `getAllEmails` to `listEmails` in EmailController and EmailService for clarity.
- Updated related tests to reflect the new method name and adjusted parameters accordingly.
- Enhanced API documentation to clarify the behavior of the `list` method, including mailbox filtering.
- Removed deprecated `getAllEmails` method from MailProvider and JmapMailProvider, consolidating email retrieval logic.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 6, 2026

@xabg2 xabg2 merged commit 21c851b into master Apr 7, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants