Passage en version 10 - Githubissues

SDIS62 / prevarisc-passerelle-platau

Passerelle permettant d'automatiser les échanges entre Prevarisc et Plat'AU

Other

9 stars 4 forks source link

Passage en version 10 #60

Closed A709197 closed 7 months ago

A709197 commented 7 months ago

Bonjour @kdubuc,

Cette PR pour passer la passerelle sur la version 10 de Plat'AU, le décommissionnement de la version 9 étant prévu pour le 31/01/2024. De mon côté, aucun BC constaté sur le périmètre de la passerelle.

J'ai profité de cette PR pour faire quelques modifications/ajouts :

Utilisation des URLs piste au lieu de aife.economie
Modification de la nomenclature pour les pièces liées aux PeC (nomenclature ajoutée en version 10)
Ajout des informations du préventionniste en charge du dossier Prevarisc si renseignées lors de l'export de la PeC et de l'avis (informations optionnelles mais fortement recommandées en version 10)

N'hésites pas à me ping si nécessaire. Cordialement, Maxime Merrien

kdubuc commented 7 months ago

Peux tu enlever les parties de code mentionnant PrevariscAuteur pour les isoler dans une PR à part ? Il faut toujours découpler Prevarisc des services Plat'AU, et là ce n'est pas le cas, ce sera plus simple de travailler dessus dans une branche. Le reste est OK pour moi.

A709197 commented 7 months ago

Quelle serait alors la méthode à suivre pour ajouter les informations de l'auteur sur les méthodes du service PlatauConsultation ? Ajouter un argument par valeur ? L'idée du ValueObject était justement de ne pas ajouter 5 arguments à ces méthodes qui en contiennent déjà un certain nombre.

kdubuc commented 7 months ago

L'idée du VO est très bien ! Mais il ne faut pas la typer Prevarisc.

Prend dans l'autre sens : type là comme un objet Plat'AU (avec les attributs Plat'AU tels que décrits dans l'openapi). Le service Prevarisc aurait une fonction getDossierAuteur() qui renvoie les informations sous forme d'array (le besoin de typage fort n'est pas vraiment fort ici), et dans la commande tu construis le VO avec ces données là (via un service translator ou directement dans la commande) qui sera utilisé par les services Platau

A709197 commented 7 months ago

J'ai ajouté la fonction qui récupère les informations de l'auteur dans le service Prevarisc et qui servent à construire le VO dans les commandes. Le VO est nommé génériquement Auteur. Je ne sais pas si ça fait l'affaire ?

kdubuc commented 7 months ago

Parfait pour le VO et pour la logique du service Prevarisc & des commandes. Je vais être tatillon : plus qu'a rendre Auteur facultatif (sur PEC et avis) et gérer ce cas d'usage, puis je merge 😉

Merci Maxime

A709197 commented 7 months ago

C'est fait !

Merci pour les retours 😃

kdubuc commented 7 months ago

Merci Maxime ! Je merge

resolve #55