typescript-angular: Add correct library versions for Angular 8.0.0

[steffen678]

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

After the issue regarding the http client has been sorted out, the only step for Angular 8 support are the missing libraries, which I have added here. If there's anything wrong, please let me know. Not sure what to do about the petshop examples.

@TiFu @taxpon @sebastianhaas @kenisteward @Vrolijkx @macjohnny @nicokoenig @topce @akehir

[macjohnny]

LGTM

[macjohnny]

please revert the default version to 7.0.0 to avoid a breaking change and have this feature released in the next release

Suggested change

	protected String ngVersion = "8.0.0";
	protected String ngVersion = "7.0.0";

[macjohnny]

@Veluria thanks for your PR!

[macjohnny]

@Veluria

Not sure what to do about the petshop examples.

please run bin/typescript-angular-petstore-all.sh and commit the changed files in the samples folder

[steffen678]

@Veluria

Not sure what to do about the petshop examples.

please run bin/typescript-angular-petstore-all.sh and commit the changed files in the samples folder

Thanks for your fast response. I did run the script, but no files were changed, I guess. :)

[macjohnny]

Thanks for your fast response. I did run the script, but no files were changed, I guess. :)

this what we would expect as the version still defaults to 7.0.0 now.
In order to allow reviewing the generated 8.0.0 specific code, you could duplicate
https://github.com/OpenAPITools/openapi-generator/blob/master/bin/typescript-angular-v7-petstore-provided-in-root-with-npm.sh

and add it to
https://github.com/OpenAPITools/openapi-generator/blob/master/bin/typescript-angular-petstore-all.sh (don't forget the windows version https://github.com/OpenAPITools/openapi-generator/blob/master/bin/windows/typescript-angular-petstore-all.bat)

moreover, you could duplicate https://github.com/OpenAPITools/openapi-generator/blob/master/samples/client/petstore/typescript-angular-v7-provided-in-root/pom.xml with the corresponding tests
and add it to
https://github.com/OpenAPITools/openapi-generator/blob/master/pom.xml#L1064

to have the version 8.0.0 client also tested by the CI

[steffen678]

Added! I was wondering about this too, but it wasn't in the previous version, so I wasn't sure if it should go there.

[wing328]

@Veluria thanks for the PR, which has been included in the v4.1.1 release: https://twitter.com/oas_generator/status/1165944867391860737