Skip to content

fix(system-cleanup): invalid trash permissions#1055

Closed
dhruvmistry2000 wants to merge 1 commit intoChrisTitusTech:mainfrom
dhruvmistry2000:patch-2
Closed

fix(system-cleanup): invalid trash permissions#1055
dhruvmistry2000 wants to merge 1 commit intoChrisTitusTech:mainfrom
dhruvmistry2000:patch-2

Conversation

@dhruvmistry2000
Copy link
Contributor

the function was not working as it didint have sudo permission

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Hotfix
  • Security patch
  • UI/UX improvement

Description

Testing

Impact

Issues / other PRs related

  • Resolves #

Additional Information

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

the function was not working as it didint have sudo permission
@dhruvmistry2000
Copy link
Contributor Author

image
The issue

@koibtw koibtw changed the title Adding sudo permission for cleanup fix(system-cleanup): invalid trash permissions Apr 1, 2025
@koibtw
Copy link
Collaborator

koibtw commented Apr 1, 2025

the changes look good. i'm not sure if this is the correct solution though. your permissions seem to be pretty messed up

@Jaredy899
Copy link
Contributor

Jaredy899 commented Apr 1, 2025

In all the testing I've done on various distros, I've not had a problem with permissions in cleanup.

@dhruvmistry2000
Copy link
Contributor Author

Ok i will check and I am closing the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants