-
Notifications
You must be signed in to change notification settings - Fork 187
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
[full-ci] Update reva to move validation to the public share provider #7877
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. |
17e3582
to
4fd7363
Compare
cb640cf
to
9c8ced9
Compare
CI is failed |
Fixed it. Let us see now. |
5cf6a60
to
0808917
Compare
0808917
to
3b56a57
Compare
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM on the testing side
[full-ci] Update reva to move validation to the public share provider
Description
This is the ocis part of cs3org/reva#4372
It makes the password policy config global.
Already existing
OCIS_SHARING_PUBLIC_SHARE_MUST_HAVE_PASSWORD
andOCIS_SHARING_PUBLIC_WRITEABLE_SHARE_MUST_HAVE_PASSWORD
have already been there
Frontend service
Sharing service
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: