-
Notifications
You must be signed in to change notification settings - Fork 159
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
Trash bin enhancements #6609
Trash bin enhancements #6609
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
Results for oCISSharingPublic1 https://drone.owncloud.com/owncloud/web/23790/67/1
|
Results for oCISTrashbinUploadMoveJourney https://drone.owncloud.com/owncloud/web/23790/69/1
|
Results for oCISSharingInternal2 https://drone.owncloud.com/owncloud/web/23809/60/1
|
Results for oCISSharingInternal3 https://drone.owncloud.com/owncloud/web/23809/61/1
|
Results for oCISFiles1 https://drone.owncloud.com/owncloud/web/23823/55/1
|
61f1493
to
b2513c1
Compare
Results for oC10IntegrationApp1 https://drone.owncloud.com/owncloud/web/23913/71/1 💥 The acceptance tests pipeline failed. The build has been cancelled. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works like a charm!
packages/web-app-files/tests/unit/components/AppBar/AppBar.spec.js
Outdated
Show resolved
Hide resolved
packages/web-app-files/tests/unit/components/AppBar/AppBar.spec.js
Outdated
Show resolved
Hide resolved
packages/web-app-files/tests/unit/components/AppBar/AppBar.spec.js
Outdated
Show resolved
Hide resolved
@@ -26,6 +26,10 @@ export default { | |||
) { | |||
return false | |||
} | |||
if (!resources.every((r) => r.canBeRestored())) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can use this logic more often. I think currently the user is being offered to perform certain batch actions that then fail somewhere down the selected items because permissions are missing 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In that special case, the permission check will be done via WebDavProperties, but not implemented yet in the backend
Kudos, SonarCloud Quality Gate passed! |
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Page title rendered correctly in space trash bin
Navigate to space action in app bar
Breadcrumb for personal trash bin
Types of changes
Checklist:
Open tasks: