Skip to content

feat: Per-request OpenAPI config overrides #1777

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: Per-request OpenAPI config overrides #1777

wants to merge 1 commit into from

Conversation

eakorolev
Copy link

@eakorolev eakorolev commented Sep 21, 2023 •
edited
Loading

Fixes #1538 #1765

@eakorolev eakorolev force-pushed the feat/config-overrides branch 6 times, most recently from 15cc452 to d204ec4 Compare September 21, 2023 14:46
@eakorolev eakorolev force-pushed the feat/config-overrides branch from d204ec4 to 6947775 Compare September 21, 2023 15:22
@mirismaili
Copy link

mirismaili commented Oct 29, 2023 •
edited
Loading

Thank @eakorolev. Although this doesn't resolve #1538 completely. See here.

Is there a way to use your fork before publish?

@mirismaili
Copy link

I published a temporary package here:

https://www.npmjs.com/package/forked-openapi-typescript-codegen

@mirismaili mirismaili mentioned this pull request Oct 29, 2023
Open
@eakorolev
Copy link
Author

@mirismaili, thank you for publishing this package! As s temporary solution, I've just installed a patched package from GitHub branch like this:

  "devDependencies": {
    "openapi-typescript-codegen": "github:eakorolev/openapi-typescript-codegen#feat/config-overrides"
  }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Possibility to pass context/options from query-executor to the client/request
2 participants
@eakorolev @mirismaili