-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Labels
good first issueIdeal for newcomers to start contributingIdeal for newcomers to start contributingtype: enhancementNew features or improvements to existing features.New features or improvements to existing features.
Description
Description
Migrate the LocalDeleteOperationDecider from the legacy module (:legacy:core) into a new LocalDeleteOperationDecider interface within the feature domain api :feature:mail:message:list:api and a default implementation inside the :feature:mail:message:list:internal.
Goal
Encapsulate the business rules that determine if a deletion should bypass the trash (e.g., when deleting from Spam/Trash or based on DeletePolicy) without bloating the DeleteMessages orchestrator.
Acceptance Criteria
- Correctly identifies "hard delete" scenarios based on
Accountsettings andFoldertype. - Ensure the logic is tested correctly.
Technical Details
- Need to move both
DefaultLocalDeleteOperationDecider(oldLocalDeleteOperationDecider) implementation AND tests to the internal feature module - Need to wire the
MessagingControllerto retrieve the instance of the new interfaceLocalDeleteOperationDecidervia DI.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueIdeal for newcomers to start contributingIdeal for newcomers to start contributingtype: enhancementNew features or improvements to existing features.New features or improvements to existing features.
Type
Projects
Status
In Review