Skip to content

typescript-fetch: fix return type of primitive value #4028

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

Merged
merged 3 commits into from
Oct 2, 2019

Conversation

ffMathy
Copy link
Contributor

@ffMathy ffMathy commented Oct 2, 2019

Fixes #3709
Fixes #2870.

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh, ./bin/openapi3/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\. If contributing template-only or documentation-only changes which will change sample output, be sure to build the project first.
  • Filed the PR against the correct branch: master, 4.1.x, 5.0.x. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

(details of the change, additional tests that have been done, reference to the issue for tracking, etc)

@ffMathy
Copy link
Contributor Author

ffMathy commented Oct 2, 2019

@TiFu (2017/07) @taxpon (2017/07) @sebastianhaas (2017/07) @kenisteward (2017/07) @Vrolijkx (2017/09) @macjohnny (2018/01) @nicokoenig (2018/09) @topce (2018/10) @akehir (2019/07)

@macjohnny
Copy link
Member

@ffMathy thanks for the PR.
please re-generate the samples with https://github.com/OpenAPITools/openapi-generator/blob/master/bin/typescript-fetch-petstore-all.sh and commit them

@macjohnny macjohnny changed the title Update apis.mustache typescript-fetch: fix return type of primitive value Oct 2, 2019
@macjohnny macjohnny added this to the 4.1.3 milestone Oct 2, 2019
@ffMathy
Copy link
Contributor Author

ffMathy commented Oct 2, 2019

How can I run this file on Windows?

@macjohnny
Copy link
Member

@ffMathy
Copy link
Contributor Author

ffMathy commented Oct 2, 2019

Running it produced no differences in the pet API code. That makes sense, since the pet API doesn't have any endpoints returning booleans or numbers.

@ffMathy
Copy link
Contributor Author

ffMathy commented Oct 2, 2019

Actually, scratch that. It should indeed produce a difference, but it doesn't. Any ideas?

@macjohnny
Copy link
Member

according to the CI (https://circleci.com/gh/OpenAPITools/openapi-generator/9216#tests/containers/2) there should be changes.
please run mvn clean package before re-generating the samples.

@ffMathy
Copy link
Contributor Author

ffMathy commented Oct 2, 2019

mvn clean package helped! Thanks.

@macjohnny macjohnny merged commit 88ed52f into OpenAPITools:master Oct 2, 2019
@ffMathy
Copy link
Contributor Author

ffMathy commented Oct 2, 2019

Thanks for merging!

How long until this is available on NPM?

@macjohnny
Copy link
Member

It will be included in the next release, so probably next week.

Jesse0Michael pushed a commit to Jesse0Michael/openapi-generator that referenced this pull request Oct 3, 2019
* Update apis.mustache

Fixes OpenAPITools#3709 and OpenAPITools#2870.

* Update apis.mustache

* added new petstore code.
@wing328
Copy link
Member

wing328 commented Oct 4, 2019

@ffMathy thanks for the PR, which has been included in the v4.1.3 release: https://twitter.com/oas_generator/status/1180123829626003456

jimschubert added a commit that referenced this pull request Oct 6, 2019
* master: (35 commits)
  [haskell-http-client] update samples (#4073)
  [haskell-http-client] Bump deps to LTS 14.7 (#4068)
  update release for 4.2.0
  [typescript-axios] Fix api generating incorrect seralization type check (#4051)
  prepare 4.1.3 release (#4052)
  typescript-node: form data file (#3967)
  Add a link to blog post on vertx and openapi (#4048)
  better wording for apiNameSuffix option description (#4045)
  [Ruby] fix ruby test, update error message (#4041)
  [PHP] Correctly format JSON in headers (#4024)
  [haskell-http-client] add dateTimeParseFormat cli option - overrides the format string used to parse a datetime (#4037)
  Add frankyjuang to the C# technical committee (#4036)
  Feature/api name suffix (#3918)
  [F#] minor improvements to the generators (#3968)
  Repaired Checkstyle (#4029)
  mockito 3.1.0 (#4035)
  typescript-fetch: fix return type of primitive value (#4028)
  [typescript][node]: Add accept header if produces is not empty (#3966)
  [haskell-http-client] disable unused import warning in Core.hs (#4020)
  Add a link to the tutorial in http4k (#4019)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants