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

feature: Typescript-angular support single request param #4479

Merged
merged 12 commits into from
Nov 14, 2019
Merged

feature: Typescript-angular support single request param #4479

merged 12 commits into from
Nov 14, 2019

Conversation

djairhogeuens
Copy link
Contributor

@djairhogeuens djairhogeuens commented Nov 13, 2019

fixes #4465 Add support for singleRequestParameter in typescript-angular generator

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.

@djairhogeuens
Copy link
Contributor Author

@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)

Please have a look :)

*/
public addPet(body: Pet, extraHttpRequestParams?: RequestOptionsArgs): Observable<{}> {
return this.addPetWithHttpInfo(body, extraHttpRequestParams)
public addPet(pet: Pet, extraHttpRequestParams?: RequestOptionsArgs): Observable<{}> {
Copy link
Member

Choose a reason for hiding this comment

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

please don't run the bin/openapi3 scripts

@macjohnny
Copy link
Member

thanks for the PR, I will have a closer look soon

@macjohnny macjohnny added this to the 4.2.1 milestone Nov 13, 2019
Copy link
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
I think we should include it and have a look at the user feedback.

@macjohnny macjohnny changed the title Typescript angular single request param feature: Typescript-angular support single request param Nov 13, 2019
@macjohnny
Copy link
Member

@djairhogeuens can you please commit the executable flag of the newly added script in /bin?

@djairhogeuens
Copy link
Contributor Author

@djairhogeuens can you please commit the executable flag of the newly added script in /bin?

The executable flag was set, I did notice a small mistake in the script itself when comparing it to the windows version. I ran them both locally and they seem to work fine so I commited that fix and will see if CI passes now.

@djairhogeuens
Copy link
Contributor Author

Hm strange... In my terminal it showed that the file was executable (and I could execute it) but when checking git ls-files --stage it said that it was not executable. I managed to fix it now :)

@filipvh
Copy link

filipvh commented Nov 14, 2019

@macjohnny @djairhogeuens Build seems to have timed out?

@macjohnny
Copy link
Member

@filipvh I restarted the build

@macjohnny
Copy link
Member

@djairhogeuens the CircleCI tests fail due to the docs-page not being up to date. can you please fix this?

@djairhogeuens
Copy link
Contributor Author

Docs were updated but apparently the order of the parameters matters, fixed it :)
The TypeScriptAngularClientOptionsProvider.java was already updated with the new option

@wing328
Copy link
Member

wing328 commented Nov 15, 2019

Thanks for the PR but your commit (as shown in the Commits tab) is not linked to your Github account, which means this PR won't count as your contribution in https://github.com/OpenAPITools/openapi-generator/graphs/contributors.

Let me know if you need help fixing it.

Ref: https://github.com/OpenAPITools/openapi-generator/wiki/FAQ#how-can-i-update-commits-that-are-not-linked-to-my-github-account

@djairhogeuens
Copy link
Contributor Author

@wing328 When checking in SourceTree it seems correct to me?
image

@wing328
Copy link
Member

wing328 commented Nov 15, 2019

@djairhogeuens thanks for the PR, which has been included in the v4.2.1 release: https://twitter.com/oas_generator/status/1195339336922759168

@wing328
Copy link
Member

wing328 commented Nov 15, 2019

@djairhogeuens please have a look at https://github.com/OpenAPITools/openapi-generator/pull/4479/commits and none of the commits is linked to your Github account.

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.

[Feature][typescript-angular] useSingleRequestParameter missing on typescript-angular generator
4 participants