-
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
Make amount of group members in patch configurable. #5357
Conversation
This PR changes the following: * Create an API config section for API configurables. * Add a setting `UserPatchLimit` that controls how many users can be changed in a PATCH request. * Use this setting in the API to limit the amount of users that can be changed.
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. |
💥 Acceptance test Core-API-Tests-ocis-storage-2 failed. Further test are cancelled... |
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.
Just a minor naming suggestion. And could you maybe add a test case for this?
Also make test for old default match on the actual error message.
ef63ad5
to
6f55e32
Compare
Kudos, SonarCloud Quality Gate passed! |
This PR changes the following:
UserPatchLimit
that controls how many users can be changed in a PATCH request.