Supprimer les données de subventions sur les CCAS (non asso)

[alice-telescoop]

https://datasubvention.beta.gouv.fr/association/265300855

Ce n'est pas une asso mais on a des données dauphin dessus

À merger après #554

On fait le cas 1 proposer par Alice !

[VSerain]

Je ne suis pas sûr qu'on puisse le considérer comme un bug ! Faudrait savoir si c'est dérengant ou pas, selon moi si pas objection juridique non on laisse comme ça !

[alice-telescoop]

Faut confirmer avec les gens de l'équipe. Ils l'ont remonté comme un bug voire un accident diplomatique. Mais je suis d'accord pas sûr que ce soit un bug non plus. Plutôt une absence de feature qui serait importante..? Je retrouve le channel mattermost

[alice-telescoop]

:/ je ne le trouve pas ç'a dû être vu à l'oral. Ç'a été remonté par Alix ou Alex

[VSerain]

Je vais re demandée

[alice-telescoop]

oups je viens de voir ton message et ça va peut-être poser pb pour les routes open-data, je fais pas de filtre sur si c'est des assos

[VSerain]

Ducoup le ticket ça serait plutot rajouter un filtre association sur les sub, voir dans notre route de sub renvoyer une erreur c'est c'est pas une asso

[alice-telescoop]

Oui je pense que ça va se finir comme ça

[alice-telescoop]

Et sur la partie open-data il faut le faire avant la mise en prod

[alice-telescoop]

processus de validation

Cas 1 :

• [x] la table Historique Unité Légale est à jour (cf #554)

flowchart 
    1["l'identifiant est un rna ?"] -->|"oui"| 2(["on garde"])
    1 -->|"non"| 941841["siren dans la table\nsiren-entreprises ?"]
    941841 -->|"oui"| 488645(["on jette"])
    941841 -->|"non"| 612618(["on garde"])

Cas 2 :

la table Historique Unité Légale n'est pas à jour

flowchart 
    1["l'identifiant est un rna ?"] -->|"oui"| 2(["on garde"])
    1 -->|"non"| 941841["siren dans la table\nsiren-entreprises ?"]
    941841 -->|"oui"| 488645(["on jette"])
    757713["rnaService.getRna()\nrenvoie un résultat ?"] -->|"non"| 725092["l'API Asso trouve\nun code juridique compatible"]
    757713 -->|"oui"| 181574(["on garde"])
    941841 -->|"non"| 757713
    725092 -->|"oui"| 874688(["on garde"])
    725092 -->|"non"| 612618(["on jette"])

[mxmeunier]

@mxmeunier créé un ticket pour n'afficher que le message "ceci n'est pas une association" et ne pas afficher les tabs

[VSerain]

Voir pour créer le ticket qui va bien

[VSerain]

@VSerain ne m'oublie pas !

[VSerain]

Voir avec Yasmine si on le fait vraiment ? Est-ce que ça vaut le coup de passer du temps de dev là-dessus par rapport au reste !

[VSerain]

Ne consomée que les asso dans notre systeme

[alice-telescoop]

On revert parce que le siren 422374728 de l'asso "VILLA NOAILLES" est mal répertorié, et ce n'est sûrement pas le seul. En attente de règles métier, cf mattermost

[VSerain]

Le soucis ne viens pas de code ape car ce n'est pas la meme chose qu'une categorie, le soucis viens d'un if chez nous, que je vais clean cf https://github.com/betagouv/api-subventions-asso/blob/develop/packages/api/src/modules/providers/datagouv/parse.uniteLegal.service.ts#L124 ça devrai etre :

if is asso : 
  if should be saved : save asso
if isnewuniteLegal:
  save  entreprise

Dansle cas actuelle ça va dans save entreprise quand c'est une asso mais que ne doit pas etre save

[VSerain]

Soucis régler dans ma pr unite legal et vu qu'on ve reparser les data ça devrai etre clean

[alice-telescoop]

Donc faudra re-appliquer la PR quand ta PR sera passée pour vérifier

[VSerain]

@alice-telescoop on à un peut oublier ce truc je crois, est-ce qu'on tente de juste re-apliquer t'a PR ?

[alice-telescoop]

On peut mais comme t'as rajouté des méthodes pour check ça, je pense qu'il faut faire le point (et ça peut être dans le ticket)

[mxmeunier]

@alice-telescoop je pense qu'on peut clôturer ce ticket, non ?

[alice-telescoop]

Pour moi ça vaudrait le coup de faire le point technique des différentes choses qu'on fait parce qu'il y a beaucoup de fonctions qui travaillent sur ça à des moments différents. J'en fait un refacto investigation qu'on prendra un jour peut-être