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

Divide "Shared with me" view into pending, accepted and declined sections #5177

Closed
wants to merge 2 commits into from

Conversation

elizavetaRa
Copy link
Member

@elizavetaRa elizavetaRa commented May 31, 2021

Description

The "Shared with me" view shows 2 different lists for pending and accepted shares. On request user can see the declined files (clicking "show declined shares).

Motivation and Context

General UX improvement: Motivate the user to accept or decline the shares through better overview of the statuses and highlighted pending section.

How Has This Been Tested?

UI test scenario: Sharing multiple files through one testing account with another

  • test case 1: pending shares, 3 or less pending shares --> simple list of pending shares
  • test case 2: pending shares, more than 3 shares --> "show all", "show less" option
  • test case 3: no pending shares--> no pending shares list shown
  • test case 4: no accepted shares--> "You are currently not collaborating on other people's resources" message shown, "show declined shares" option appears at pending shares
  • test case 5: accepted shares exist--> list with accepted shares and option to show declined shares appears
  • test case 6: no declined shares--> "No declined files found" message appears
  • test case 7 declined shares exist --> list with declined shares shown on click of "show declined shares"

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

General view

Bildschirmfoto vom 2021-05-31 10-14-23

@update-docs
Copy link

update-docs bot commented May 31, 2021

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.

@ownclouders
Copy link
Contributor

💥 Acceptance tests oc10-integration-notifications failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15842/

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests webUISharingNotifications failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15842/

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests IntegrationApp1 failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15842/

webUISharingAcceptSharesToRoot-acceptShares-feature-50.png

webUISharingAcceptSharesToRoot-acceptShares-feature-50.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests webUISharingBasic failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15842/

webUISharingAcceptShares-acceptShares-feature-100.png

webUISharingAcceptShares-acceptShares-feature-100.png

webUISharingAcceptShares-acceptShares-feature-135.png

webUISharingAcceptShares-acceptShares-feature-135.png

webUISharingAcceptShares-acceptShares-feature-16.png

webUISharingAcceptShares-acceptShares-feature-16.png

webUISharingAcceptShares-acceptShares-feature-203.png

webUISharingAcceptShares-acceptShares-feature-203.png

webUISharingAcceptShares-acceptShares-feature-235.png

webUISharingAcceptShares-acceptShares-feature-235.png

webUISharingAcceptSharesToRoot-acceptShares-feature-110.png

webUISharingAcceptSharesToRoot-acceptShares-feature-110.png

webUISharingAcceptSharesToRoot-acceptShares-feature-131.png

webUISharingAcceptSharesToRoot-acceptShares-feature-131.png

webUISharingAcceptSharesToRoot-acceptShares-feature-143.png

webUISharingAcceptSharesToRoot-acceptShares-feature-143.png

webUISharingAcceptSharesToRoot-acceptShares-feature-15.png

webUISharingAcceptSharesToRoot-acceptShares-feature-15.png

webUISharingAcceptSharesToRoot-acceptShares-feature-158.png

webUISharingAcceptSharesToRoot-acceptShares-feature-158.png

webUISharingAcceptSharesToRoot-acceptShares-feature-267.png

webUISharingAcceptSharesToRoot-acceptShares-feature-267.png

webUISharingAcceptSharesToRoot-acceptShares-feature-299.png

webUISharingAcceptSharesToRoot-acceptShares-feature-299.png

webUISharingAcceptSharesToRoot-acceptShares-feature-31.png

webUISharingAcceptSharesToRoot-acceptShares-feature-31.png

webUISharingAcceptSharesToRoot-acceptShares-feature-315.png

webUISharingAcceptSharesToRoot-acceptShares-feature-315.png

webUISharingAcceptSharesToRoot-acceptShares-feature-357.png

webUISharingAcceptSharesToRoot-acceptShares-feature-357.png

webUISharingAcceptSharesToRoot-acceptShares-feature-366.png

webUISharingAcceptSharesToRoot-acceptShares-feature-366.png

webUISharingAcceptSharesToRoot-acceptShares-feature-397.png

webUISharingAcceptSharesToRoot-acceptShares-feature-397.png

webUISharingAcceptSharesToRoot-acceptShares-feature-50.png

webUISharingAcceptSharesToRoot-acceptShares-feature-50.png

webUISharingAcceptSharesToRoot-acceptShares-feature-90.png

webUISharingAcceptSharesToRoot-acceptShares-feature-90.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests oc10-integration-notifications failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15843/

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests webUISharingNotifications failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15843/

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests IntegrationApp1 failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15843/

webUISharingAcceptSharesToRoot-acceptShares-feature-50.png

webUISharingAcceptSharesToRoot-acceptShares-feature-50.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests webUISharingBasic failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15843/

webUISharingAcceptShares-acceptShares-feature-100.png

webUISharingAcceptShares-acceptShares-feature-100.png

webUISharingAcceptShares-acceptShares-feature-135.png

webUISharingAcceptShares-acceptShares-feature-135.png

webUISharingAcceptShares-acceptShares-feature-16.png

webUISharingAcceptShares-acceptShares-feature-16.png

webUISharingAcceptShares-acceptShares-feature-203.png

webUISharingAcceptShares-acceptShares-feature-203.png

webUISharingAcceptShares-acceptShares-feature-235.png

webUISharingAcceptShares-acceptShares-feature-235.png

webUISharingAcceptSharesToRoot-acceptShares-feature-110.png

webUISharingAcceptSharesToRoot-acceptShares-feature-110.png

webUISharingAcceptSharesToRoot-acceptShares-feature-131.png

webUISharingAcceptSharesToRoot-acceptShares-feature-131.png

webUISharingAcceptSharesToRoot-acceptShares-feature-143.png

webUISharingAcceptSharesToRoot-acceptShares-feature-143.png

webUISharingAcceptSharesToRoot-acceptShares-feature-15.png

webUISharingAcceptSharesToRoot-acceptShares-feature-15.png

webUISharingAcceptSharesToRoot-acceptShares-feature-158.png

webUISharingAcceptSharesToRoot-acceptShares-feature-158.png

webUISharingAcceptSharesToRoot-acceptShares-feature-267.png

webUISharingAcceptSharesToRoot-acceptShares-feature-267.png

webUISharingAcceptSharesToRoot-acceptShares-feature-299.png

webUISharingAcceptSharesToRoot-acceptShares-feature-299.png

webUISharingAcceptSharesToRoot-acceptShares-feature-31.png

webUISharingAcceptSharesToRoot-acceptShares-feature-31.png

webUISharingAcceptSharesToRoot-acceptShares-feature-315.png

webUISharingAcceptSharesToRoot-acceptShares-feature-315.png

webUISharingAcceptSharesToRoot-acceptShares-feature-357.png

webUISharingAcceptSharesToRoot-acceptShares-feature-357.png

webUISharingAcceptSharesToRoot-acceptShares-feature-366.png

webUISharingAcceptSharesToRoot-acceptShares-feature-366.png

webUISharingAcceptSharesToRoot-acceptShares-feature-397.png

webUISharingAcceptSharesToRoot-acceptShares-feature-397.png

webUISharingAcceptSharesToRoot-acceptShares-feature-50.png

webUISharingAcceptSharesToRoot-acceptShares-feature-50.png

webUISharingAcceptSharesToRoot-acceptShares-feature-90.png

webUISharingAcceptSharesToRoot-acceptShares-feature-90.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests XGAPortrait1-Notifications failed. The build is cancelled...

1 similar comment
@ownclouders
Copy link
Contributor

💥 Acceptance tests XGAPortrait1-Notifications failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests oc10-integration-notifications failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/16040/

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotifications-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-40.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

webUISharingNotificationsToRoot-shareWithUsers-feature-53.png

💥 Acceptance tests oc10-integration-notifications failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests XGAPortrait1-Notifications failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests SharingFilePermission failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests XGAPortrait1 failed. The build is cancelled...

@ownclouders
Copy link
Contributor

💥 Acceptance tests XGAPortrait1-Notifications failed. The build is cancelled...

@elizavetaRa
Copy link
Member Author

@kulmann I rebased the pr but we again face the problem with updating the list by accepting/declining #5263 We also updated the server to 1.7.0, so it could be a server problem.

@kulmann kulmann mentioned this pull request Jun 19, 2021
5 tasks
@ownclouders
Copy link
Contributor

💥 Acceptance tests XGAPortrait1 failed. The build is cancelled...

@kulmann
Copy link
Member

kulmann commented Jun 21, 2021

Hey @elizavetaRa, I already rebased to our current master and wanted to work on the PR in #5314 (opened a new PR since I can't push to CERN repos) - what kind of changes did you apply, except for rebasing?

@kulmann
Copy link
Member

kulmann commented Sep 21, 2021

Thank you for your contribution! We finished the PR in #5814, closing this PR here as the functionality is in master now.

@kulmann kulmann closed this Sep 21, 2021
@diocas diocas deleted the owncloud branch October 25, 2021 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants