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

Also show owner and resharer for indirect shares #2918

Merged
merged 1 commit into from
Jan 29, 2020

Conversation

PVince81
Copy link
Contributor

@PVince81 PVince81 commented Jan 27, 2020

Description

Display collaborator entries for indirect shares (inherited from parent resources) for owner and resharer.

Related Issue

Follow up of #2915
Fixes #2898

Motivation and Context

How Has This Been Tested?

  • visual check in collaborators panel
  • acceptance tests for the new cases

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

Checklist:

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

Open tasks:

  • Fix avatar for owner and resharer
  • Display "via" label
  • Use "redo" icon instead of anchor Use "exit to app"
  • TEST: unskip and adjust acceptance tests
  • TEST: extend some tests to also include "Resharer" entry
  • TEST: add test for "shared with me"
  • add ellipsis on via text

@PVince81
Copy link
Contributor Author

Co-authored and co-reviewed with @kulmann.

@PVince81
Copy link
Contributor Author

likely blocked by #2927

Co-authored-by: Vincent Petry <pvince81@owncloud.com>
Co-authored-by: Benedikt Kulmann <benedikt@kulmann.biz>

- Introduced npm dependency for path handling
- Added via label above of owner and resharer for indirect shares
- Refactor collaborator actions into the collaborator vue component.
- Restyle collaborator list to use horizontal dividers.
- Move content of collaborator list item from a grid into a table
- Added test for via label
- Added test for owner display in "shared with me" page
- Refactored getCollaboratorsList to be able to return more detailed information
@PVince81 PVince81 force-pushed the share-indirect-info-continued branch from 28dbb46 to 239616f Compare January 29, 2020 11:00
@PVince81
Copy link
Contributor Author

Rebased to get the avatar fix.

Conflict: had to integrate the <avatar-image> and shareTypes constant usage changes in the markup of Collaborator.vue.

Retested, still works.

@PVince81 PVince81 merged commit 20f5913 into master Jan 29, 2020
@PVince81
Copy link
Contributor Author

🎆

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

Successfully merging this pull request may close these issues.

See resource owner in share panel
3 participants