Skip to content

Conversation

bvwells
Copy link
Contributor

@bvwells bvwells commented Mar 17, 2018

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 and ./bin/security/{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\.
  • Filed the PR against the correct branch: 3.0.0 branch for changes related to OpenAPI spec 3.0. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.
    @TiFu @taxpon @sebastianhaas @kenisteward @Vrolijkx @macjohnny

Description of the PR

Make optional properties in models optional parameters in generated typescript-node client. Addresses issue #7791.

@bvwells
Copy link
Contributor Author

bvwells commented Mar 17, 2018

Seems to be an issue with the reliability of some circleci tests, #7862

@bvwells
Copy link
Contributor Author

bvwells commented Mar 18, 2018

Resubmission of #7849.

@bvwells bvwells closed this Mar 28, 2018
@bvwells bvwells reopened this Mar 28, 2018
@wing328 wing328 added this to the v2.4.0 milestone Mar 30, 2018
@bvwells
Copy link
Contributor Author

bvwells commented Mar 30, 2018

After discussion with @wing328, I'ved added the package.json in the swagger codegen ignore file so that the package.json won't be overwritten with re-generating the typescript node client. This is because the package file has been modified to add dependencies required for testing the client (specifically rewire).

@wing328 wing328 merged commit c8650d0 into swagger-api:master Mar 31, 2018
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.

3 participants