Projet

Général

Profil

Development #67326

toulouse-maelis: ajout des endpoints pour créer/modifier les responsables légaux

Ajouté par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
13 juillet 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

...


Fichiers

0001-toulouse_maelis-parsifal-add-webservices-to-manage-R.patch (39,5 ko) 0001-toulouse_maelis-parsifal-add-webservices-to-manage-R.patch Nicolas Roche (absent jusqu'au 3 avril), 13 juillet 2022 00:14
0005-toulouse_maelis-relax-constaint-on-family-schema-673.patch (1,21 ko) 0005-toulouse_maelis-relax-constaint-on-family-schema-673.patch Nicolas Roche (absent jusqu'au 3 avril), 18 juillet 2022 01:50
0004-toulouse_maelis-modify-coordinate-schema-desciption-.patch (1,37 ko) 0004-toulouse_maelis-modify-coordinate-schema-desciption-.patch Nicolas Roche (absent jusqu'au 3 avril), 18 juillet 2022 01:50
0003-toulouse_maelis-add-test-on-updateFamily-error-67326.patch (6,44 ko) 0003-toulouse_maelis-add-test-on-updateFamily-error-67326.patch Nicolas Roche (absent jusqu'au 3 avril), 18 juillet 2022 01:50
0002-toulouse_maelis-adapt-payloads-on-RL-tests-67326.patch (8,8 ko) 0002-toulouse_maelis-adapt-payloads-on-RL-tests-67326.patch Nicolas Roche (absent jusqu'au 3 avril), 18 juillet 2022 01:50
0005-toulouse_maelis-relax-constaint-on-family-schema-673.patch (4,87 ko) 0005-toulouse_maelis-relax-constaint-on-family-schema-673.patch Nicolas Roche (absent jusqu'au 3 avril), 20 juillet 2022 18:42
0004-toulouse_maelis-modify-coordinate-schema-desciption-.patch (1,36 ko) 0004-toulouse_maelis-modify-coordinate-schema-desciption-.patch Nicolas Roche (absent jusqu'au 3 avril), 20 juillet 2022 18:42
update_family_test.py (2,13 ko) update_family_test.py Nicolas Roche (absent jusqu'au 3 avril), 20 juillet 2022 18:42

Demandes liées

Lié à Passerelle - Development #66880: toulouse-maelis: ajout des endpoints permettant de créer et de modifier les DUI (ticket chapeau)Fermé01 juillet 2022

Actions

Révisions associées

Révision c238a9f2 (diff)
Ajouté par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an

toulouse_maelis: parsifal: add webservices to manage RL (#67326)

Historique

#1

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an

On n'a rien pour supprimer le RL2, mais ça ne choque pas Stéphane.

#2

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é
#3

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 ?

#4

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.

#5

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an

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.

J'ajoute plusieurs commits, mais ce ne sont que des ajustements sans conséquence :
  • 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.
#6

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'])
#7

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a plus d'un an

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).

#10

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)

#11

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)

#12

Mis à jour par Transition automatique il y a plus d'un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#13

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF