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

show file-locking information #6682

Closed
exalate-issue-sync bot opened this issue Mar 28, 2022 · 7 comments · Fixed by #9566 or #9658
Closed

show file-locking information #6682

exalate-issue-sync bot opened this issue Mar 28, 2022 · 7 comments · Fixed by #9566 or #9658
Assignees
Labels

Comments

@exalate-issue-sync
Copy link

exalate-issue-sync bot commented Mar 28, 2022

Description

Info about locking is (somehow) available in the backend

User Stories

  • As a user I who has write permissions to a file I want to know if a file is locked, so that I know why I can't edit it.

Value

  • working on office files

Acceptance Criteria

  • If file is locked, 🔒-Icon is shown next to share-indicators in filelist and Details-Right Sidebar Panel
  • Icon is shown immeadiately (--> SSE)-> split out to new ticket
  • Click on 🔒-Icon in files-list, opens right sidebar "Details" -Panel
  • "Lock"Information in the files-details:
    • If Locked Show "Lock: Locked by [Displayname] 23 minutes ago." (if this info is available)
    • If Locked Show "Locked since: Relative Date" (+ Tooltip with Absolute Date)
    • If Locked Show "Locked expires: Relative Date" (+ Tooltip with Absolute Date)
  • Automatic unlocking happens after 30 mins per default; can be adjusted via config file.
  • If file is locked: not available fileactions (move, rename, delete) are disabled and get the addition "(locked)" see sribble
  • Cherry on top: add an indicator on the Filetype icon, so that users see instantly if a file is locked.

Definition of done

  • Functional requirements
    [ ] functionality described in the user story works
    [ ] acceptance criteria are fulfilled
  • Quality
    [ ] codre review happened
    [ ] CI is green
    [ ] critical code received unit tests by the developer
    [ ] automated tests passed (if automated tests are not available, this test needs to be created and passed
  • Non-functional requirements
    [ ] no sonar cloud issues
@exalate-issue-sync
Copy link
Author

Benedikt Kulmann commented: locking information is available in propfinds via property {DAV:}lockdiscovery. Needs to be added to the default dav properties in the constants of packages/web-pkg.

@tbsbdr
Copy link
Contributor

tbsbdr commented Jul 26, 2023

closed by mistake, reopening

@lookacat
Copy link
Contributor

we can't get the timestamp of the lock since its not implemented in the backend

@exalate-issue-sync
Copy link
Author

Florian Schade commented: SSE part follows: owncloud/ocis#6992

@tbsbdr
Copy link
Contributor

tbsbdr commented Aug 22, 2023

As discussed:

  • should be accessible
  • Add tippy "Item is locked" on hover of the filetype&lock-icon
  • single-selection: disable non-available actions (cut, rename, delete)

Note: lets see if the overlayed lock-icon on the file-icon receives positive feedback by the users; if not we might need to move it to the shared/status column. let's try :D

@lookacat
Copy link
Contributor

done, will only add unittests then it can get merged

@ScharfViktor
Copy link
Contributor

that is great, we avoid a lot of errors when user tries to delete or move locked file

one question: maybe also hide buttons create tag and share for locked file? because it fails like deleting or moving locked file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants