[REQ][typescript-angular] Angular 9 providedIn support

[UnleashSpirit]

Hello,
Angular 9 brings new level for @Injectable, 'any' and 'platform'

Is your feature request related to a problem? Please describe.

No

Describe the solution you'd like

Removing the 'providedInRoot' option as boolean and gives a new one.
Something like 'providedIn' with all the choices :

• none
• root
• any
• platform

May be with keeping default as root ?
And may be a ApiModule.forPlatform() ? (don't even know if it makes sense)

Additional context

I face that 'issue', which is not a real one for now more an improvement, during Angular Elements creation (WebComponent).
In my case I have a service provided un 'platform' level which also depend on generated service from openapi v3. But a 'platform' service can't use a root level service (null injector)

In any case I think that supporting the new levels of Angular 9 Injectable would be appreciate

[macjohnny]

@UnleashSpirit would you like to implement this?

[UnleashSpirit]

Waow hmm yeah I can try.
Don't know when but I could be fun :)
How it works ? Fork then modify then pull request right ?

[macjohnny]

Waow hmm yeah I can try.

great

How it works ? Fork then modify then pull request right ?

exactly, here are some starting points:

openapi-generator/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/TypeScriptAngularClientCodegen.java

Lines 99 to 101 in b87c927

	this.cliOptions.add(CliOption.newBoolean(PROVIDED_IN_ROOT,
	"Use this property to provide Injectables in root (it is only valid in angular version greater or equal to 6.0.0).",
	false));

openapi-generator/modules/openapi-generator/src/main/resources/typescript-angular/api.service.mustache

Lines 46 to 53 in b87c927

	{{^providedInRoot}}
	@Injectable()
	{{/providedInRoot}}
	{{#providedInRoot}}
	@Injectable({
	providedIn: 'root'
	})
	{{/providedInRoot}}

[jogelin]

@UnleashSpirit did you progress on this one ?

[UnleashSpirit]

Hi, unfortunately not yet :(
Trying to find time but hard these days