New collaborators flow#2450
Conversation
|
Todos:
|
💥 Acceptance tests RestrictSharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests SharingInternalGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests Files failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests SharingPermissionsUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests SharingInternalUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests Resharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests iPhone failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests XGAPortrait failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests SharingPermissionsGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6370/
|
💥 Acceptance tests RestrictSharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests SharingInternalGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests XGAPortrait failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests Files failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests SharingInternalUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests iPhone failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests SharingPermissionsUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests Resharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests SharingPermissionsGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6417/
|
💥 Acceptance tests RestrictSharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6420/
|
💥 Acceptance tests SharingInternalGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6420/
|
💥 Acceptance tests SharingInternalUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6420/
|
💥 Acceptance tests SharingPermissionsUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6420/
|
💥 Acceptance tests Resharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6420/
|
|
the change collaborator button does not change the mouse when h-over |
known general issue with icons: owncloud/owncloud-design-system#488 |
4e28b79 to
c4980eb
Compare
|
Rebased and adjusted tests based on comments, except for the one about the autocomplete "no results" case |
💥 Acceptance tests RestrictSharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
💥 Acceptance tests SharingFilePermissionsGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
💥 Acceptance tests SharingInternalUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
💥 Acceptance tests SharingPermissionsUsers failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
💥 Acceptance tests SharingInternalGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
💥 Acceptance tests XGAPortrait failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
💥 Acceptance tests iPhone failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
💥 Acceptance tests Resharing failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
💥 Acceptance tests SharingFolderPermissionsGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6699/
|
c4980eb to
507040c
Compare
|
I messed up the names. Fixed now. Expecting all tests to pass. Then need approval for the tests from @individual-it and a code review from @LukasHirt |
💥 Acceptance tests SharingFilePermissionsGroups failed. Please find the screenshots inside ...https://drone.owncloud.com/owncloud/phoenix/6701/
|
|
tests look good. some are failing I guess because there is a new suite since #2531 and |
|
Yeah, basically every day I fix tests and rebase, new tests appear on master so I need to fix again 😢 |
Editing and deleting colalborators is now happening inside a separate dialog instead of a series of accordions. Fixed updating share in the state. Adjusted acceptance tests for properly opening the dialog before asserting or operating on roles or permissions.
507040c to
1431ad6
Compare
|
Nope, this time I had it in my local workspace and somehow failed to commit it... Pushed now. Thanks for the review. @LukasHirt code review ? |
|
npm fail for "Notifications" test 😞 |
|
since the notifications test is unrelated and passed the last time, I'll override and merge anyway |




































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































































Description
Related Issue
Fixes #1908
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: