Skip to content

Conversation

@yashwanthgajji
Copy link
Contributor

@yashwanthgajji yashwanthgajji commented Dec 6, 2024

Description

Added an option in behavior settings for delete permanently check box. When enabled, it disables the functionality of trash bin and delete files permanently.

Delete Permanently Checkbox image

Issue tracker

Fixes #4277

Automatic tests

  • Added test cases

Manual tests

  • Done

Build tasks success

Successfully running following tasks on local:

  • ./gradlew assembledebug
  • ./gradlew spotlessCheck

@VishnuSanal VishnuSanal added the PR-Awaiting-Initial-Review this PR is awaiting for an initial review label Dec 13, 2024
Copy link
Member

@VishnuSanal VishnuSanal left a comment

Choose a reason for hiding this comment

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

hi, thanks for your interest in contributing to amaze! please see the comments for reviews.

@VishnuSanal VishnuSanal added PR-Requested-Changes this PR is awaiting an update from the author and removed PR-Awaiting-Initial-Review this PR is awaiting for an initial review labels Dec 13, 2024
Copy link
Member

@VishnuSanal VishnuSanal left a comment

Choose a reason for hiding this comment

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

@yashwanthgajji some more changes:

please add the info that this disables the trashbin to the description.
also, we should still show the confirmation dialog if "delete confirmation" is checked. the current implementation essentially revokes that previous setting!
thanks!

VishnuSanal

This comment was marked as outdated.

Copy link
Member

@VishnuSanal VishnuSanal left a comment

Choose a reason for hiding this comment

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

LGTM, great work, keep going! :) 🚀

@VishnuSanal VishnuSanal added PR-Awaiting-Final-Review this PR is awaiting a final review/approval and removed PR-Requested-Changes this PR is awaiting an update from the author labels Dec 19, 2024
@EmmanuelMess EmmanuelMess merged commit 16d068e into TeamAmaze:release/4.0 May 14, 2025
3 checks passed
@EmmanuelMess EmmanuelMess removed the PR-Awaiting-Final-Review this PR is awaiting a final review/approval label May 14, 2025
@EmmanuelMess EmmanuelMess mentioned this pull request May 25, 2025
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.

Option for single click to delete permanently

3 participants