search

Syspeople / CodeNow

Empower your ServiceNow Development with no instance configuration.
MIT License
8 stars 7 forks source link

Add Angular Providers #87

Closed CGraabaek closed 5 years ago

CGraabaek commented 5 years ago

Add support fro angular providers.

CGraabaek commented 5 years ago

@ambsoerensen jeg har lavet det her i branchen AddAngularProvider, har tjekket den nye awesome contribution guide ud og alt spiller bare.

Har et spørgsmål til en enkelt ting jeg mangler at implementere.

Hvordan skal en "custom" template til Records implementeres? En Angular Provider har nemlig en type, "Service, Directive, Factory" som skal vælges når man opretter en.

Lige nu virker det fint uden, men så defaulter den bare til Directive.

Giver dette mening?

ambsoerensen commented 5 years ago

pt. ligger det i instance klassen. https://github.com/Syspeople/snsb/blob/master/src/ServiceNow/Instance.ts#L1193

Overvejet at flytte det ud men blev enig med mig selv om det kunne vente til en eventuelt custom template funktionalitet.

Der definere du bare et objekt der bruges til at oprette med :)

CGraabaek commented 5 years ago

Yes har fået styr på template-systemet, mega nice!

Men tænkte mere at det her var en custom ting ifht. at man skal have lave en QuickPicker hvor man kan vælge typen.

ambsoerensen commented 5 years ago

Umiddelbart tror jeg det bliver en switch lidt ligesom template tingen i instance.ts

Da vi skal styre ui etc. Giver det vidst bedst mening at smide det i extension.ts i create commanden. Vi har samme behov med script includes og dens "client callable" option.

ambsoerensen commented 5 years ago

96