Development #67326
toulouse-maelis: ajout des endpoints pour créer/modifier les responsables légaux
0%
Description
...
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an
- Fichier 0001-toulouse_maelis-parsifal-add-webservices-to-manage-R.patch 0001-toulouse_maelis-parsifal-add-webservices-to-manage-R.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
On n'a rien pour supprimer le RL2, mais ça ne choque pas Stéphane.
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an
- Lié à Development #66880: toulouse-maelis: ajout des endpoints permettant de créer et de modifier les DUI (ticket chapeau) ajouté
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Ton patern il me me semble qu'il y manque ^/$ pour ne pas matcher les chaînes qui ont ce préfixe (je crois que le souci est généralisé dans le code de passerelle).
La lecture du RL via son id en base plutôt que de dire simplement RL1/RL2 en ignorant son numéro c'est ce qui a été trouvé de plus logique ?
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an
Ton patern il me me semble qu'il y manque ^/$ pour ne pas matcher les chaînes qui ont ce préfixe
Bien vu, merci.
La lecture du RL via son id en base (...) c'est ce qui a été trouvé de plus logique ?
Oui, c'est pour rester cohérent avec les enfants (et peut-être aussi les personnes autorisées),
où l'on connaîtra l'identifiant de l'enfant depuis le slug de la page combo.
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an
- Fichier 0005-toulouse_maelis-relax-constaint-on-family-schema-673.patch 0005-toulouse_maelis-relax-constaint-on-family-schema-673.patch ajouté
- Fichier 0004-toulouse_maelis-modify-coordinate-schema-desciption-.patch 0004-toulouse_maelis-modify-coordinate-schema-desciption-.patch ajouté
- Fichier 0003-toulouse_maelis-add-test-on-updateFamily-error-67326.patch 0003-toulouse_maelis-add-test-on-updateFamily-error-67326.patch ajouté
- Fichier 0002-toulouse_maelis-adapt-payloads-on-RL-tests-67326.patch 0002-toulouse_maelis-adapt-payloads-on-RL-tests-67326.patch ajouté
- Statut changé de Solution validée à Solution proposée
La lecture du RL via son id en base (...) c'est ce qui a été trouvé de plus logique ?
En fait je pourrais aussi avoir la place de l'enfant dans la liste comme slug de la sous-page combo
(et pour les vaccins je vais effectivement devoir utiliser leur place dans la liste parce qu'ils n'ont pas d'identifiant).
Donc non, pas vraiment de logique, juste que je préfères identifier les RL, enfants et personnes par leur identifiant que par leur place.
- 0002: modif des payload des tests, parce qu'au début j'étais parti sur les blocs de champs
- 0003: pour garder en mémoire qu'un bug maélis m'empêche de récupérer les codes d'erreurs
(je laisse le test car s'il y a correction côté maélis, il améliorera la couverture de code).
https://redmine.sigec.fr/issues/931 - 0004: juste du blabla
- 0005: je n'avais pas compris qu'on pouvait utiliser updateFamily sans passer aucun responsable légal ni enfant.
Mis à jour par Thomas Noël il y a plus d'un an
Sur le premier patch, c'est quand même un peu moche d'avoir du mélange de phrases avec ou pas i18n, d'une ligne à l'autre, genre
display_category=_('Family'), description="Informations sur un responsable légal",
ou
'NameID': {'description': _('Publik ID')}, 'rl_id': {'description': 'Numéro du représentant légal'},
mais bon, comme le reste est ainsi déjà écrit, let's go :)
Sur les autres :- 0002: ok
- 0003 : je n'ai pas compris d'où vient l'erreur « Unexpected element 'adresse', expected 'lastname' » du test... c'est pas le XML envoyé qui est foireux ? Je m'attendais à ce que le test montre qu'on a reçu un message de UPDATE_FAMILY_ERR
- 0004 : ligne 308, mette un commentaire et pas une docstring
- 0005 : "updateFamily sans passer aucun responsable légal ni enfant" : ok mais j'imagine que pour la création c'est quand même nécessaire. Il faut garder ce FAMILY_SCHEMA pour create_family, et créer un UPDATE_FAMILY_SCHEMA pour update_family (genre copy.deepcopy de FAMILY_SCHEMA et del['Oneof'])
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an
- Fichier 0005-toulouse_maelis-relax-constaint-on-family-schema-673.patch 0005-toulouse_maelis-relax-constaint-on-family-schema-673.patch ajouté
- Fichier 0004-toulouse_maelis-modify-coordinate-schema-desciption-.patch 0004-toulouse_maelis-modify-coordinate-schema-desciption-.patch ajouté
- Fichier update_family_test.py update_family_test.py ajouté
Sur le premier patch, c'est quand même un peu moche ...
Oui, j'ai ajouté un patch au ticket chapeau pour retirer les phrases génériques que j'ai laissées. (#67565)
0003 : je n'ai pas compris d'où vient l'erreur « Unexpected element 'adresse', expected 'lastname' » du test... c'est pas le XML envoyé qui est foireux ? Je m'attendais à ce que le test montre qu'on a reçu un message de UPDATE_FAMILY_ERR
Tout juste, à la base le test est prévu pour montrer qu'on a reçu un message d'erreur de Maélis.
Sauf que la trame renvoyée est refusée par zeep ; j'ai ouvert un ticket sur le redmine Sigec pour avoir une correction (https://redmine.sigec.fr/issues/931).
Je joins un script pour reproduire.
J'ai laissé le test parce qu'une fois le correctif déployé par Sigec il passera dans le code de traitement d'erreur, que j'ai préféré laisser bien qu'aujourd'hui ce soit du code mort.
Tu préfères que je retire tout ça ?
0004 : fait
0005 : j'imagine que pour la création c'est quand même nécessaire.
Oui, tu as raison (pour la création il faut obligatoiremet avoir RL1).
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Ack
(je n'ai pas bien compris 0003 mais je te fais confiance)
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
(squashé)
commit c238a9f2b06668959c7c3c1691618025630a758e Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Tue Jul 12 18:56:47 2022 +0200 toulouse_maelis: parsifal: add webservices to manage RL (#67326)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
toulouse_maelis: parsifal: add webservices to manage RL (#67326)