-
Notifications
You must be signed in to change notification settings - Fork 27
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
🎨 web-server api: ordering parameters and simplified openapi specs for complex query parameters #6737
🎨 web-server api: ordering parameters and simplified openapi specs for complex query parameters #6737
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6737 +/- ##
==========================================
- Coverage 88.33% 85.70% -2.63%
==========================================
Files 1440 1573 +133
Lines 58398 63309 +4911
Branches 1405 2125 +720
==========================================
+ Hits 51583 54257 +2674
- Misses 6615 8739 +2124
- Partials 200 313 +113
Continue to review full report in Codecov by Sentry.
|
82d2ea8
to
7827e0d
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.
Very nice refactoring.
This is the inverse of parse_as_body/header/param/query methods. nice!
Just a few comments and questions.
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.
Thanks so much! It’s really nice. I’ve left a few comments.
2b9b63b
to
911f54e
Compare
911f54e
to
357ba30
Compare
357ba30
to
d85312a
Compare
Quality Gate passedIssues Measures |
What do these changes do?
Unifies and simplifies ordering queries. For instance, list
folders
is displayed asfrom this coded openapi specification
♻️ Unified
ordering
query parameterspackages/models-library/src/models_library/rest_ordering.py
$(ResourceName)$(OperationName)OrderQueryParams
(e.g.,FoldersListOrderQueryParams
).♻️ Simplified handling of complex query parameters (i.e. JSON-encoded)
as_query
for ease of useapi/specs/web-server
to reduce complexity and potential errors.. E.g. openapi specs are simply defined using one model per "group" of parameters_p
/ query_q
) parameters.Related issue/s
How to test
Driving test
Dev-ops
None