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

Implement dynamic roles for server 10.3+ and oCIS #7487

Closed
HanaGemela opened this issue Sep 24, 2019 · 4 comments
Closed

Implement dynamic roles for server 10.3+ and oCIS #7487

HanaGemela opened this issue Sep 24, 2019 · 4 comments
Labels
Enhancement feature:sharing p2-high Escalation, on top of current planning, release blocker
Milestone

Comments

@HanaGemela
Copy link
Contributor

Background is here https://github.com/owncloud/enterprise/issues/3473

Steps to recreate:

  1. Connected to server 10.3+ and oCIS
  2. Right click on a resource
  3. Select Share from ownCloud submenu
  4. Select Public links

Expected result: Roles are displayed instead of permissions

@pmaier1
Copy link

pmaier1 commented Sep 24, 2019

Expected result: Roles are displayed instead of permissions

One clarification: It is expected that the client makes use of the new (10.3) roles API (if the capability is present) in order to fetch the available permissions/roles (for public links and user/group sharing) from the server and to display them dynamically.

The main reasons that the roles API was implemented are

  • We added a new role for public links. Clients can't implement it statically because its availability depends on the server they connect to (https://github.com/owncloud/enterprise/issues/3464).
  • Since we have "Advanced Sharing Permissions" (10.2) server extensions may add permissions to user/group sharing. Implementing those in the clients has the same challenges as above.

This is also necessary for Android (owncloud/android#2667) and iOS (owncloud/ios-app#506).

@michaelstingl
Copy link
Contributor

@ogoffart
Copy link
Contributor

ogoffart commented Feb 6, 2020

Note that there is a POC for the hybrid approach:
#7564

@michaelstingl
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement feature:sharing p2-high Escalation, on top of current planning, release blocker
Projects
None yet
Development

No branches or pull requests

4 participants