Skip to content

Fix: Save all option overrides attachment files#10967

Open
mohitsatr wants to merge 1 commit intothunderbird:mainfrom
mohitsatr:attachment-overrriden
Open

Fix: Save all option overrides attachment files#10967
mohitsatr wants to merge 1 commit intothunderbird:mainfrom
mohitsatr:attachment-overrriden

Conversation

@mohitsatr
Copy link
Copy Markdown
Contributor

@mohitsatr mohitsatr commented May 1, 2026

Fixes #10965

Changes:

  1. Makes use of OpenDocumentTree Contract onSaveAllAttachments to let users choose a destination folder for all the attachments. Previously, we were prompted to click 'save' for each attachment.
  2. Convert AttachmentController.java to Kotlin to make use of Coroutines.

Behaviour:

Screen_Recording_20260506_085902_Thunderbird.Debug.mp4

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

Missing report label. Set exactly one of: report: include, report: exclude OR report: highlight.

@mohitsatr mohitsatr force-pushed the attachment-overrriden branch from 09cfb0a to f76dfdb Compare May 5, 2026 12:29
@mohitsatr mohitsatr force-pushed the attachment-overrriden branch from f76dfdb to 8de676d Compare May 6, 2026 03:30
@mohitsatr mohitsatr marked this pull request as ready for review May 6, 2026 03:44
@mohitsatr mohitsatr requested a review from a team as a code owner May 6, 2026 03:44
@mohitsatr mohitsatr requested a review from dani-zilla May 6, 2026 03:44
@mohitsatr mohitsatr changed the title (WIP) Fix: Save all option overrides attachment files Fix: Save all option overrides attachment files May 6, 2026
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.

More than 1 attached file overridden by the option "Save all"

2 participants