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

Check for SetProjectSpaceQuota permission #3690

Merged
merged 1 commit into from
Mar 16, 2023

Conversation

kobergj
Copy link
Contributor

@kobergj kobergj commented Feb 27, 2023

Check Drive.ReadWriteQuota.Project when editing the quota of a project space instead of set-spaces-quota.

part of the fix for owncloud/ocis#5475

DONT MERGE - NEEDS A CASCADE OF PR MERGINGs

@kobergj kobergj force-pushed the CheckSetProjectSpaceQuotaPermission branch 2 times, most recently from f1a2f4a to eed3cfa Compare February 27, 2023 10:19
@kobergj kobergj changed the title Check set-project-spaces-quota permission Check for SetProjectSpaceQuota permission Feb 27, 2023
@kobergj kobergj marked this pull request as draft February 27, 2023 11:33
@kobergj
Copy link
Contributor Author

kobergj commented Feb 27, 2023

Needs a web client change first, otherwise it is no longer possible to change quotas through the UI

Don't merge update

@kobergj kobergj force-pushed the CheckSetProjectSpaceQuotaPermission branch from eed3cfa to 4a614d2 Compare March 15, 2023 15:23
Signed-off-by: jkoberg <jkoberg@owncloud.com>
@kobergj kobergj force-pushed the CheckSetProjectSpaceQuotaPermission branch from 4a614d2 to 6b25288 Compare March 15, 2023 15:29
@kobergj kobergj marked this pull request as ready for review March 15, 2023 15:56
@kobergj kobergj merged commit 3c11349 into cs3org:edge Mar 16, 2023
@kobergj kobergj deleted the CheckSetProjectSpaceQuotaPermission branch March 16, 2023 15:47
@micbar micbar mentioned this pull request May 3, 2023
89 tasks
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.

2 participants