Skip to content

Changes to permissions with recursive operations#298

Merged
TimCsaky merged 1 commit intomasterfrom
recursive-delete
Apr 8, 2025
Merged

Changes to permissions with recursive operations#298
TimCsaky merged 1 commit intomasterfrom
recursive-delete

Conversation

@TimCsaky
Copy link
Contributor

@TimCsaky TimCsaky commented Apr 7, 2025

Recursive delete requires DELETE perm on all sub-folders
Recursive Sync copies current user's permissions to all sub-folders

Description

Types of changes

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@TimCsaky TimCsaky requested review from norrisng-bc and removed request for norrisng-bc April 7, 2025 20:03
@github-actions
Copy link

github-actions bot commented Apr 7, 2025

Coverage Report

Totals Coverage
Statements: 57.09% ( 3057 / 5355 )
Methods: 47.18% ( 334 / 708 )
Lines: 63.79% ( 1832 / 2872 )
Branches: 50.2% ( 891 / 1775 )

@TimCsaky TimCsaky force-pushed the recursive-delete branch 3 times, most recently from beb6559 to 941ead2 Compare April 7, 2025 23:55
Recursive delete requires DELETE perm on all sub-folders
Recursive Sync copies current user's permissions to all sub-folders
@TimCsaky TimCsaky merged commit 71aa37a into master Apr 8, 2025
13 checks passed
@TimCsaky TimCsaky deleted the recursive-delete branch April 29, 2025 21:03
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.

2 participants