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

first prototype for the permissions service api #161

Merged
merged 2 commits into from
Dec 14, 2021

Conversation

C0rby
Copy link
Contributor

@C0rby C0rby commented Dec 8, 2021

This is the proposed CS3 API extension to introduce a permissions service.
Currently this API is read-only but eventually should also allow to set or update permissions / assignments.

@C0rby C0rby mentioned this pull request Dec 8, 2021
@C0rby C0rby marked this pull request as ready for review December 10, 2021 14:09
@C0rby C0rby requested a review from labkode as a code owner December 10, 2021 14:09
@refs
Copy link
Member

refs commented Dec 10, 2021

@ishank011 could you review this one please?

@refs
Copy link
Member

refs commented Dec 13, 2021

@C0rby rebase? 👼

@refs
Copy link
Member

refs commented Dec 13, 2021

Reva implementation lives in cs3org/reva#2341

@C0rby C0rby force-pushed the permissions-service branch 3 times, most recently from acad4b0 to 5caea34 Compare December 13, 2021 12:12
@C0rby C0rby marked this pull request as draft December 13, 2021 12:13
@C0rby
Copy link
Contributor Author

C0rby commented Dec 13, 2021

Ah, I messed something up. There are changes in this PR which are not from me. I'll fix that and then it should be good to go.

@C0rby
Copy link
Contributor Author

C0rby commented Dec 13, 2021

I don't really know why this is here: https://github.com/cs3org/cs3apis/pull/161/files#diff-b04b38d4e36f7a7171aeb211bf933eaf36d41d9866ebbc3639f673f84dc350aeL5710-R5906.

Rebasing didn't remove it. I don't find any place where I am touching this in the proto files..

@C0rby C0rby marked this pull request as ready for review December 13, 2021 12:22
@labkode labkode merged commit 48b68f8 into cs3org:main Dec 14, 2021
@C0rby C0rby deleted the permissions-service branch December 14, 2021 10:29
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.

3 participants