Skip to content
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

Add listFromJson method to fix a bug #5355

Merged
merged 1 commit into from May 2, 2020
Merged

Conversation

selankon
Copy link
Contributor

@selankon selankon commented Feb 17, 2020

Fix #4887

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.

@wing328
Copy link
Member

wing328 commented Feb 18, 2020

cc @ircecho (2017/07) @swipesight (2018/09) @jaumard (2018/09) @nickmeinhold (2019/09) @athornz (2019/12) @amondnet (2019/12)

@wing328
Copy link
Member

wing328 commented Feb 20, 2020

We'll merge it over the weekend if there's no further feedback from anyone.

@wing328 wing328 added this to the 4.3.0 milestone Feb 20, 2020
@nickmeinhold
Copy link
Contributor

Awesome work with the issue and repro steps! You've set everything up to make it easy to add a test for this to the CI - would you be up for that? No worries if not, I can add a new issue to create tests for this and some other recent PRs.

@selankon
Copy link
Contributor Author

Thanks (:
I'm mot sure of how to create the tests, maybe if you do it I can check the code an learn it for the future for future PR. Meanwhile I'm little busy now...

@wing328
Copy link
Member

wing328 commented May 1, 2020

I'll merge it over the weekend. Sorry for taking a bit long.

@wing328 wing328 merged commit 6439990 into OpenAPITools:master May 2, 2020
@wing328 wing328 changed the title Add listFromJson method to fix #4887 Add listFromJson method to fix a bug May 6, 2020
michaelpro1 pushed a commit to michaelpro1/openapi-generator that referenced this pull request May 7, 2020
MikailBag pushed a commit to MikailBag/openapi-generator that referenced this pull request May 31, 2020
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][DART] Method listFromJson isn't defined on array of enums
4 participants