Closed tschaffter closed 3 years ago
I propose to swap the parameter properties
name
andin
so thatname
comes first
Sounds good to me. I think Swagger lists them like this as well.
@vpchung Swagger doc didn't adopt a consistent behavior (example: https://swagger.io/specification/)
lol well that's awesome....
Fixes #123
Notes
list
path, I added two query parameters namedsort
anddirection
using the same parameter name and schema as in GitHub API. This information will be used by the API service to sort results before returning paginated responses.name
andin
so thatname
comes first. This is because one is probably more interested in the parameter name, then the type of the parameter. This order is also more consistent with traditional schema where each property have their name on the first line. Example:Before:
After:
If this proposal is accepted, this change will be propagated to all other path parameters in a separate PR.