-
Notifications
You must be signed in to change notification settings - Fork 248
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
Expand support for parameter serialization #1078
Conversation
Size Change: +12.1 kB (+0.58%) Total Size: 2.08 MB
ℹ️ View Unchanged
|
Visit the preview URL for this PR (updated for commit 93f8794): https://docusaurus-openapi-36b86--pr1078-tm9k8x09.web.app (expires Sat, 08 Mar 2025 18:49:47 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: bf293780ee827f578864d92193b8c2866acd459f |
Thanks Bryan, is the plan to, eventually, add support for the other style/parameters as defined here? |
Hi @sserrata, we should definitely aim to cover all parameter types and serializing combinations. We can look further into supporting all combinations in a separate PR? |
Path params test results (cURL): q = "openapi"
tags = ["api", "docs"]
filters = {"minPrice": 10, "maxPrice": 100}
sort = ["name", "price"]
fields = ["id", "name", "description"]
|
Thanks for covering the rest of the params! Everything looks good on my end 👍 |
Description
This PR consists of adding support for
explode
and correct example generation for array query parameters.Motivation and Context
See #1024 for context
How Has This Been Tested?
Tested with OpenAPI example spec provided in #1024.
Screenshots (if appropriate)
Types of changes
Checklist