-
Notifications
You must be signed in to change notification settings - Fork 189
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
enforce set space quota permission #2650
Conversation
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. |
0b85e1e
to
99fb935
Compare
💥 Acceptance tests localApiTests-apiSpaces-ocis failed. The build is cancelled... |
@C0rby Before your PR, the list spaces response was returning the Displayname of the User for the personal space. Now it is returning "root" |
this is the problem https://github.com/cs3org/reva/pull/2162/files#diff-d308a0558595437a537e77e29346472e66ab3bc31c213fcd2dd9b5f64c02aa91R348-R357 You brought back old behavior that was already changed |
Ah then I must have lost that while rebasing... |
I'll open another PR in reva... |
💥 Acceptance tests localApiTests-apiSpaces-ocis failed. The build is cancelled... |
4315ded
to
1d48fef
Compare
1d48fef
to
c23071f
Compare
Kudos, SonarCloud Quality Gate passed! |
Description
This PR enforces the set-space-quota permission on the update storage space /update drive request.
Before merging this we need: cs3org/reva#2162 in reva first.
Types of changes
Checklist: