search

infra-geo-ouverte / igo2-lib

Librairie Infrastructure Géomatique Ouverte 2.0 (IGO-2) / Open GIS Infrastructure 2.0 Library
https://infra-geo-ouverte.github.io/igo2-lib/
40 stars 26 forks source link

Check if app_initializer is still required #1558

Open pelord opened 11 months ago

pelord commented 11 months ago

Please tell us about your environment:

Check if, in the demo and in igo2, it's still required to setup an app initilizer.

{ provide: APP_INITIALIZER, useFactory: appInitializerFactory, deps: [Injector, ApplicationRef], multi: true } This checkup is required due to the merge of this PR. https://github.com/infra-geo-ouverte/igo2-lib/pull/1551

pelord commented 11 months ago

A mon avis, c'Est pas nécessaire.. mais il faudrait tester sur des poste - performants. Vous aviez eu le problème lors de la mise en prod de vigilance.

pelord commented 11 months ago

J'ai validé dans le constructeur de l'appmodule le service était bien la.

alecarn commented 10 months ago

J'ai amélioré le comportement dans la PR des Standalones via deux commits (LIB et Assemblage) https://github.com/infra-geo-ouverte/igo2-lib/pull/1556/commits/225f808ee5a09aea52ae9f11dfa1fc2dda060fae https://github.com/infra-geo-ouverte/igo2/pull/1119/commits/a971afe166b5ddce5cb3e745e3f0aa52b75de30e

Via le provideRootTranslation, on lie le TranslateModule de ngx-translate avec notre LanguageService. Celui-ci sera instancier à l'initialisation de l'application et s'occupe aussi d'instancier le TranslateService de ngx-translate

alecarn commented 10 months ago

Finalement, pour enlever le AppInitializerFactory cela nécessitait des changements sur le chargement de nos fichiers de translations. On règle du même coup, un problème de double téléchargements des fichiers de traductions. https://github.com/infra-geo-ouverte/igo2-lib/pull/1563