Construire la raison sociale pour les sociétés unipersonnelles directement à l'import

signaux-faibles / opensignauxfaibles

Plateforme de détection des entreprises fragiles

https://beta.gouv.fr/startup/signaux-faibles.html

MIT License

8 stars 1 forks source link

Construire la raison sociale pour les sociétés unipersonnelles directement à l'import #10

Open pierrecamilleri opened 4 years ago

pierrecamilleri commented 4 years ago

Pour l'instant, la raison sociale est lue dans le champs "denominationUniteLegale", et le fait que ce champ ne soit pas rempli dans les fichiers tests pourrait révéler une erreur. Cf PR #9.

Si ce n'est pas le cas, il faut compléter le fichier test.

pierrecamilleri commented 4 years ago

C'est bon, c'est le fichier test qui n'était pas représentatif des entreprises importées. Une ligne avec denominationUniteLegale a été rajoutée dans le fichier test, cf #8.

Par contre, ce n'est pas intuitif d'insérer en base une raison sociale vide pour les sociétés unipersonnelles, pour ensuite reconstruire une raison sociale dans chacun des map-reduce pour les calculs de variables et l'interface. Il serait logique de faire cette opération directement à l'import.

adrienjoly commented 3 years ago

@JazzyPierrot Afin de m'aider à comprendre dans quel champ est populée la raison sociale et à partir de quoi, peux-tu m'indiquer dans quelle fonction JS/TS du map-reduce c'est actuellement fait, stp ?

adrienjoly commented 3 years ago

Réponse de Pierre:

la raison sociale est parfois construite à partir des noms de dirigeants par reduce (js/common/raison_sociale.ts)
proposition: générer cette raison sociale au moment de l'import ?