-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: improve api-docs #40
Conversation
6a9c3da
to
35da7fc
Compare
35da7fc
to
56416cb
Compare
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.
LGMT :)
description: | ||
'Use only allowed properties separated by semicolon; default is ascending created_at; prefix name with hyphen/minus sign to get descending order', | ||
'Only whitelisted properties separated by semicolon; prefix with -/+ for DESC/ASC order', | ||
example: '-created_at;filename', |
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.
this example doesn't work, and when using endpoints we need to remove this string in swagger, should it still be added to documentantion?
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.
This example shows just the format for the filter. As described above - only whitelisted properties are allowed for filtering. If proper ones used it does work.
Due to lack of examples provided for swagger module some tools may later generate incorrect sample data.
This PR adds some missing properties for some decorators to resolve this issue.