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
Specify the parameters of FederatedFileSharing controllers #36214
Specify the parameters of FederatedFileSharing controllers #36214
Conversation
Shouldn't the parameters also be documented in the comment with explanations? |
c7ca1b4
to
36e5a37
Compare
Did that, just repushed because I forgot it on the first method
Not sure I'm fully qualified to provide this. |
Explanations could be copied from https://github.com/cs3org/OCM-API/blob/develop/spec.yaml |
It also doesn't matter too much right now, I'd just copy them later |
Closing for now as it's the wrong endpoint |
This is relevant now. I reopened because I would have just created the same PR again. |
36e5a37
to
7819975
Compare
Signed-off-by: Joas Schilling <coding@schilljs.com>
7819975
to
bf4c5bd
Compare
Summary
In order to get closer to OpenAPI all controllers should be up to the state of art how to defining the parameters so they are documented.
Checklist