diff --git a/CHANGELOG.md b/CHANGELOG.md index 6d128a0..96dd4b5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -18,6 +18,7 @@ and this project adheres to ### Fixed +- 🐛(dimail) fix imported mailboxes should be enabled instead of pending #659 - ⚡️(api) add missing cache for stats endpoint ## [1.10.0] - 2025-01-21 diff --git a/src/backend/mailbox_manager/tests/test_utils_dimail_client.py b/src/backend/mailbox_manager/tests/test_utils_dimail_client.py index 7a60dfd..6493710 100644 --- a/src/backend/mailbox_manager/tests/test_utils_dimail_client.py +++ b/src/backend/mailbox_manager/tests/test_utils_dimail_client.py @@ -156,6 +156,7 @@ def test_dimail_synchronization__synchronize_mailboxes(mock_warning): mailbox = models.Mailbox.objects.get() assert mailbox.local_part == "oxadmin" + assert mailbox.status == enums.MailboxStatusChoices.ENABLED assert imported_mailboxes == [mailbox_valid["email"]] diff --git a/src/backend/mailbox_manager/utils/dimail.py b/src/backend/mailbox_manager/utils/dimail.py index f7bde05..d2c2f08 100644 --- a/src/backend/mailbox_manager/utils/dimail.py +++ b/src/backend/mailbox_manager/utils/dimail.py @@ -337,10 +337,10 @@ class DimailAPIClient: last_name=dimail_mailbox["surName"], local_part=address.username, domain=domain, - secondary_email=dimail_mailbox[ - "email" - ], # secondary email is mandatory. Unfortunately, dimail doesn't + secondary_email=dimail_mailbox["email"], + # secondary email is mandatory. Unfortunately, dimail doesn't # store any. We temporarily give current email as secondary email. + status=enums.MailboxStatusChoices.ENABLED, ) imported_mailboxes.append(str(mailbox)) else: