before-interop / anomalieAdresse

Ce protocole permet le traitement d'une demande de création ou de modification d'adresses immeuble dans les IPE
https://before-interop.github.io/anomalieAdresse/
1 stars 6 forks source link

Ecart entre spec et swagger sur attribut immeubleNeuf #66

Closed ericjacq92 closed 4 months ago

ericjacq92 commented 5 months ago

Remontée Bytel : Dans la spec word, la définition de l'attribut ImmeubleNeuf indique trois possibilités : 0/1/2 ==> 0=Non ; 1=Oui (avec cuivre) ; 2=Oui (ZLIN=sans cuivre)

Dans le swagger on a défini un attributbuilding.newConstructionde type booléen qui ne permet pas de distinguer les cas "avec cuivre" et "sans cuivre". Peut on transformer le type booléen en texte énuméré pour couvrir les 3 cas possibles.

alexisthethe commented 5 months ago

Axione: OK

JeanChristopheLacroix commented 5 months ago

ORANGE : NOK c’est une donnée propre à l’OI, l’OC n’en n’est pas maitre et ne l’a pas forcément (données IPE « SansCuivre » facultative)

ggrebert commented 5 months ago

@JeanChristopheLacroix Quoi qu'il en soit, il y a quand même une erreur: Si c'est une donnée propre à l'OI, la propriété devrait être en read-only

Je vous laisse en opérateur décider qui a raison

JeanChristopheLacroix commented 5 months ago

Ce que l'OC doit préciser : si l'IMB est neuf ou non ==> ne doit donc PAS être en read_only Ce que l'OC ne peut pas préciser : si l'IMB est avec ou sans Cuivre (car la donnée est propre à l'OI, et est facultative dans l'IPE)

ggrebert commented 5 months ago

Donc je fais quoi ? Il n'y a aucune notion de Zlin dans le protocole actuellement. C'est peut être de document word qui est faux

ericjacq92 commented 5 months ago

BYTEL : Ma demande était liée à la mise en conformité par rapport à la spec word du protocole, et à la demande initiale de mon métier qui a bien prévu de pousser ce niveau de granularité :

image

Suite retour métier chez nous : on peut basculer sur un simple booléen : oui /non, il faut juste remettre la spec word interop en phase pour que ce soit bien clair pour tout le monde.

JeanChristopheLacroix commented 4 months ago

Décision Interop du 16/05 : à clôturer => rester sur 2 valeurs 0/1