Params - add style & explode parameter for deepObject/… encoding#9867
Params - add style & explode parameter for deepObject/… encoding#9867commonism wants to merge 1 commit intofastapi:masterfastapi/fastapi:masterfrom
Conversation
736dcf5 to
601b40d
Compare
|
It would be very helpful :) |
|
@commonism |
|
The deepObject encoding is not compatible to any other encoding. |
|
This would indeed be helpful. Thank you for the PR! |
|
I rebased the changes. The failure in the CI is related to the documentation build. Not sure what is causing this. |
|
Unfortunate this hasn't been accepted. It would really simplify accepting type safe complex objects in the route. |
|
Just had a look on re-basing. |
This comment was marked as outdated.
This comment was marked as outdated.
6dd6305 to
98877ee
Compare
This comment was marked as outdated.
This comment was marked as outdated.
|
Rebased |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
d53f38c to
f6261e2
Compare
This comment was marked as resolved.
This comment was marked as resolved.
f6261e2 to
096938d
Compare
8a27ae1 to
26c4a99
Compare
This comment was marked as resolved.
This comment was marked as resolved.
|
This pull request has a merge conflict that needs to be resolved. |
d345801 to
8dc3f28
Compare
|
This pull request has a merge conflict that needs to be resolved. |
allows deepObject encoded Query parameters
|
This pull request has a merge conflict that needs to be resolved. |
Hi,
this PR basically picks up the idea by @adriangb using ` … = Query(style="deepObject") for models in query parameters.
While it currently only serves deepObject for query parameters, it could do the job for all Parameters - depending on the encoding styles/explode combinations supported,
https://spec.openapis.org/oas/v3.1.0#style-examples
Questions -
This does deepObject.
This is a clone of #9768 which got closed when deleting a branch.
fixes: #3163