Skip to content

TF-4392 [Team Mailbox] Edit email in Drafts folder as new in the composer#4419

Open
dab246 wants to merge 1 commit intomasterfrom
bugfix/tf-4392-team-mailbox-save-to-team-draft-folder
Open

TF-4392 [Team Mailbox] Edit email in Drafts folder as new in the composer#4419
dab246 wants to merge 1 commit intomasterfrom
bugfix/tf-4392-team-mailbox-save-to-team-draft-folder

Conversation

@dab246
Copy link
Copy Markdown
Member

@dab246 dab246 commented Mar 26, 2026

Issue

#4392

User story 2

AS a team mailbox member
GIVEN I edit an email with a team mailbox member identity
WHEN I close my composer
THEN it is saved in the team mailbox Draft folder

Summary by CodeRabbit

  • Bug Fixes
    • Fixed draft mailbox detection to work reliably across all mailbox types.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3d962b4b-81ed-4418-8e16-f1a9f6c0be9b

📥 Commits

Reviewing files that changed from the base of the PR and between c17782b and 36b7090.

📒 Files selected for processing (1)
  • model/lib/extensions/presentation_mailbox_extension.dart

Walkthrough

The isDrafts getter in PresentationMailboxExtension is modified to handle personal and non-personal mailboxes differently. For personal mailboxes, it retains the existing role comparison logic. For non-personal (team) mailboxes, a new isDraftsTeamMailbox helper method is introduced that checks if the mailbox is a child of team mailboxes and compares the lowercase mailbox name against the lowercase drafts role constant.

Suggested labels

customer

Suggested reviewers

  • tddang-linagora
  • hoangdat
  • chibenwa
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly references the main change: team mailbox drafts handling in the composer. It directly describes the key feature being implemented (editing emails in team mailbox drafts).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bugfix/tf-4392-team-mailbox-save-to-team-draft-folder

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

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

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.

1 participant