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

Read quota from opaque #6187

Merged
merged 1 commit into from
Apr 28, 2023
Merged

Read quota from opaque #6187

merged 1 commit into from
Apr 28, 2023

Conversation

butonic
Copy link
Member

@butonic butonic commented Apr 28, 2023

this saves a getQuota trip to the storage when the quota information is already present in the space opaque properties

makes use of cs3org/reva#3828 but can be merged without it

Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@update-docs
Copy link

update-docs bot commented Apr 28, 2023

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.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

11.8% 11.8% Coverage
0.0% 0.0% Duplication

@butonic butonic merged commit 4bfda61 into master Apr 28, 2023
@delete-merged-branch delete-merged-branch bot deleted the read-quota-from-opaque branch April 28, 2023 13:55
ownclouders pushed a commit that referenced this pull request Apr 28, 2023
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
fschade pushed a commit that referenced this pull request Jul 10, 2023
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants