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

feat: Implement handling for lock managers who can unlock files #206

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Jan 3, 2024

Fixes #52

  • Allow unlock for
    • The file owner
    • Users who have permissions to manage ACL on groupfolders
      • Needs upstream fix and testing as groupfolders on master branch currently does not fetch the webdav properties
    • A allow list for a dedicated group of users who are considered to be allowed to manually remove a lock
      • Needs a UI to set this

@juliusknorr juliusknorr added enhancement New feature or request 2. developing labels Jan 3, 2024
@Jerome-Herbinet
Copy link
Member

Hi @juliushaertl,
Will your PR :

  • give more rights to unlock files to the mentioned users
  • add an admin page (in settings) to list locked files, with an unlock button for each remaining locked file
    Thanks by advance

@Mika-6-9
Copy link

Hi.
Very very good idea, if possible see and take into account the synchronization client (Nextcloud Desktop https://github.com/nextcloud/desktop) which automatically locks and unlocks Office files (Word/Excel/Powerpoint)
Thanks

@Mika-6-9
Copy link

Hi
Do you have any idea when the rest of this PR will be implemented?
Now, the file owner can force the unlocking resulting from a client, when will the possibility for an admin or even better a delegation of power be available so that we can unlock for others?
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Admin needs to unlock files
3 participants