Skip to content
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: add object with deepObject style in query parameters #2631

Merged
merged 8 commits into from
Nov 5, 2024

Conversation

maraino
Copy link
Contributor

@maraino maraino commented Aug 2, 2024

Support for objects with deepObject style in TryIt

This PR supports using objects with the deepObject style in the TryIt component.

A parameter id with the value {"role": "admin", "firstName": "Alex"} will create the query parameters id[role]=admin&id[firstName]=Alex.

This PR completes the table in PR #2231 with:

style explode URI template Primitive value id=5 Array id=[3, 4, 5] Object id={"role": "admin", "firstName": "Alex"} Covered by this PR
deepObject true n/a n/a n/a n/a /users?id[role]=admin&id[firstName]=Alex

This commit adds support for using objects with the deepObject style in
the TryIt component.

A parameter `id` with the value `{"role": "admin", "firstName": "Alex"}`
will create the query parameters `id[role]=admin&id[firstName]=Alex`.
@maraino maraino requested a review from a team as a code owner August 2, 2024 00:42
@maraino maraino requested a review from mallachari August 2, 2024 00:42
Copy link

netlify bot commented Aug 2, 2024

Deploy Preview for stoplight-elements ready!

Name Link
🔨 Latest commit a2f5768
🔍 Latest deploy log https://app.netlify.com/sites/stoplight-elements/deploys/672a030a371bf90008970342
😎 Deploy Preview https://deploy-preview-2631--stoplight-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Aug 2, 2024

Deploy Preview for stoplight-elements-demo ready!

Name Link
🔨 Latest commit a2f5768
🔍 Latest deploy log https://app.netlify.com/sites/stoplight-elements-demo/deploys/672a030af0f1670008e1febe
😎 Deploy Preview https://deploy-preview-2631--stoplight-elements-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mnaumanali94 mnaumanali94 removed the request for review from mallachari August 9, 2024 11:31
@mnaumanali94 mnaumanali94 enabled auto-merge (squash) August 9, 2024 13:53
@mnaumanali94 mnaumanali94 added the t/to-review To be reviewed by the team label Aug 16, 2024
@mnaumanali94 mnaumanali94 merged commit c4fc263 into stoplightio:main Nov 5, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t/to-review To be reviewed by the team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants