Skip to content

typescript-fetch: Base Path can't be set to empty string#5305

Merged
macjohnny merged 2 commits intoOpenAPITools:masterfrom
dev-johnny-gh:master
Feb 13, 2020
Merged

typescript-fetch: Base Path can't be set to empty string#5305
macjohnny merged 2 commits intoOpenAPITools:masterfrom
dev-johnny-gh:master

Conversation

@dev-johnny-gh
Copy link
Copy Markdown
Contributor

@dev-johnny-gh dev-johnny-gh commented Feb 13, 2020

fixes #5282, allows developer to set base_path to an empty string.

The code check the this.configuration.basePath, if it's NOT A NULL OR UNDEFINED, then use this.configuration.basePath, otherwise use the BASE_PATH

PR checklist

  • Read the contribution guidelines.
  • If contributing template-only or documentation-only changes which will change sample output, build the project before.
  • Run the shell script(s) under ./bin/ (or Windows batch scripts under.\bin\windows) to update Petstore samples related to your fix. This is important, as CI jobs will verify all generator outputs of your HEAD commit, and these must match the expectations made by your contribution. You only need to run ./bin/{LANG}-petstore.sh, ./bin/openapi3/{LANG}-petstore.sh if updating the code or mustache templates for a language ({LANG}) (e.g. php, ruby, python, etc).
  • File the PR against the correct branch: master, 4.3.x, 5.0.x. Default: master.
  • Copy the technical committee to review the pull request if your PR is targeting a particular programming language.

Copy link
Copy Markdown
Member

@macjohnny macjohnny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@macjohnny macjohnny added this to the 4.3.0 milestone Feb 13, 2020
@macjohnny macjohnny changed the title fix #5282: Base Path can't be set to '<empty string>' typescript-fetch: Base Path can't be set to empty string Feb 13, 2020
@macjohnny macjohnny merged commit 5a5c3db into OpenAPITools:master Feb 13, 2020
MikailBag pushed a commit to MikailBag/openapi-generator that referenced this pull request Mar 23, 2020
@wing328
Copy link
Copy Markdown
Member

wing328 commented Mar 27, 2020

@duxiaofeng-github thanks for the PR, which has been included in the 4.3.0 release: https://twitter.com/oas_generator/status/1243455743937789952

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] [typescript-fetch] Base Path can't be set to '<empty string>'

3 participants