Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Option for delete permanently on single click #4290

Open
wants to merge 5 commits into
base: release/4.0
Choose a base branch
from

Conversation

yashwanthgajji
Copy link

@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.

app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-awaiting-final-review this PR is awaiting a final review/approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option for single click to delete permanently
2 participants