close
Skip to content

refactor(message-list): move LocalDeleteOperationDecider to feature module#10532

Merged
dani-zilla merged 1 commit into
thunderbird:mainfrom
timurgilfanov:issue-10515
Mar 3, 2026
Merged

refactor(message-list): move LocalDeleteOperationDecider to feature module#10532
dani-zilla merged 1 commit into
thunderbird:mainfrom
timurgilfanov:issue-10515

Conversation

@timurgilfanov
Copy link
Copy Markdown
Contributor

Summary

  • Extract LocalDeleteOperationDecider interface into :feature:mail:message:list:api
  • Move implementation as DefaultLocalDeleteOperationDecider to :feature:mail:message:list:internal
  • Wire DI through FeatureMessageListModule instead of controllerModule
  • Move tests to internal module, add StubLocalDeleteOperationDecider for MessagingControllerTest

Closes #10515

…odule

Migrate LocalDeleteOperationDecider from legacy/core to
feature/mail/message/list as an interface (api) with
DefaultLocalDeleteOperationDecider implementation (internal).
Wire DI through FeatureMessageListModule.

Fixes thunderbird#10515
Copy link
Copy Markdown
Contributor

@dani-zilla dani-zilla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@dani-zilla dani-zilla merged commit 254237f into thunderbird:main Mar 3, 2026
14 checks passed
@thunderbird-botmobile
Copy link
Copy Markdown
Contributor

Thanks for your contribution! Your pull request has been merged and will be part of Thunderbird 17. We appreciate the time and effort you put into improving Thunderbird. If you haven’t already, you’re welcome to join our Matrix chat for contributors. It’s where we discuss development and help each other out. https://matrix.to/#/#tb-android-dev:mozilla.org
Hope to see you there! 🚀📱🐦

@thunderbird-botmobile thunderbird-botmobile Bot added this to the Thunderbird 17 milestone Mar 3, 2026
@wmontwe wmontwe added the report: exclude Exclude changes from user-facing reports (internal, minor, or not relevant to users). label Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

report: exclude Exclude changes from user-facing reports (internal, minor, or not relevant to users).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ML 0.5/MVI.02.4] Implement LocalDeleteOperationDecider (Domain Logic)

4 participants