Skip to content

feat(objectParams): use single object params option #396

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

Closed

Conversation

pivaszbs
Copy link

Status

Ready

Description

A few sentences describing the overall goals of the pull request's commits.

Related PRs

List related PRs against other branches:

branch PR
other_pr_production link
other_pr_master link

Todos

  • Tests
  • Documentation
  • Changelog Entry (unreleased)

Steps to Test or Reproduce

Outline the steps to test or reproduce the PR here.

> git pull --prune
> git checkout <branch>
> grunt jasmine

@vercel
Copy link

vercel bot commented May 18, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @anymaniax on Vercel.

@anymaniax first needs to authorize it.

@yekver
Copy link
Contributor

yekver commented May 18, 2022

Seems like this PR is linked to #281

@pivaszbs pivaszbs changed the title feat(objectParams): use single object params option feat(objectParams): use single object params option (#281) May 18, 2022
@pivaszbs pivaszbs changed the title feat(objectParams): use single object params option (#281) feat(objectParams): use single object params option May 18, 2022
@anymaniax
Copy link
Collaborator

Hello @pivaszbs, thanks for the pull request. Wanted to do it for a while will need to take a bit of time to review this. Probably next week.

@anymaniax
Copy link
Collaborator

can be introduced with v7 #434 and with your own template or I would prefer an option in the config to add this to all the current template

@anymaniax anymaniax force-pushed the master branch 2 times, most recently from e2edc19 to 60b251d Compare September 23, 2022 12:55
@jrozbicki
Copy link

@anymaniax option in config similar to openapi-typescript-codegen --useOptions would be awesome! By the looks of it, this seems to be last missing piece blocking us from moving to orval

@Marcel-G
Copy link
Contributor

useNamedParameters option has been added: #914

@melloware melloware closed this Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants