Bug #72563
toulouse-maelis: prévoir d'appeler le WS get-direct-debit-order depuis combo
0%
Description
En gros, juste sortir les paramètres du corps de la requête.
Steph a mis une maquette ici :
https://portail-parsifal.test.entrouvert.org/facture/
https://redmine.sigec.fr/issues/1491
Fichiers
Demandes liées
Révisions associées
toulouse-maelis: apply direct debit order WS on RL1 (#72563)
toulouse-maelis: pass get-direct-debit order parameters in QS (#72563)
Historique
Mis à jour par Nicolas Roche il y a plus d'un an
- Lié à Development #72524: toulouse-maelis: intégrer la livraison du 14/12/2022 ajouté
Mis à jour par Nicolas Roche il y a plus d'un an
- Lié à Development #72288: toulouse-maelis: (Faturation) endpoints pour lire écrire les coordonnées bancaires ajouté
Mis à jour par Robot Gitea il y a plus d'un an
- Statut changé de Nouveau à Solution proposée
Nicolas Roche (nroche) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/passerelle/pulls/15
- Titre : utils: add date type to endpoint parameters (#72641)
- Modifications : https://gitea.entrouvert.org/entrouvert/passerelle/pulls/15/files
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier 0003-toulouse-maelis-pass-get-direct-debit-order-paramete.patch 0003-toulouse-maelis-pass-get-direct-debit-order-paramete.patch ajouté
- Fichier 0002-toulouse-maelis-apply-direct-debit-order-WS-on-RL1-7.patch 0002-toulouse-maelis-apply-direct-debit-order-WS-on-RL1-7.patch ajouté
- Fichier 0001-toulouse-maelis-modify-regie-number-on-functest-7256.patch 0001-toulouse-maelis-modify-regie-number-on-functest-7256.patch ajouté
- Patch proposed changé de Non à Oui
Mauvaise pull-request (je ne sais pas quelle est la mauvaise manipulation que j'ai faite).
J'ai posé 2 branches, dont une basée #72641.
Je pose ici les patchs de la branche indépendante (nommée -v2).
En fait il faut faire un peu plus, parce qu'il faut cibler le RL1 pour simplifier la lecture, puis par symétrie, également sur l'écriture.
(sur la page des factures, on n'aura pas le numéro du RL sous la main et on n'attachera pas les prélèvements automatique sur le RL2)
0001: Corrige le code régie pour les tests fonctionnels sur la recette.
0002: Modifie les appels (add et get) pour cibler le RL1
0003: Passe les paramètres du get dans la query-string.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Robot Gitea il y a plus d'un an
- Statut changé de Solution validée à Solution proposée
Mis à jour par Robot Gitea il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne (bdauvergne) a approuvé une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/passerelle/pulls/15
- Commentaire :
Ça me parait bien. À pousser vendredi.
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier 0004-toulouse-maelis-adapt-after-rebase-on-72638-to-fixup.patch 0004-toulouse-maelis-adapt-after-rebase-on-72638-to-fixup.patch ajouté
- Statut changé de Solution validée à Solution proposée
J'ajoute un dernier patch qui sera requis après rebase sur #72638 qui améliore le nouveau code de test SOAP.
Mis à jour par Nicolas Roche il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
(mais comme c'est juste pour réparer les tests, je me permet de garder la validation)
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Solution proposée
Faut revoir complètement la fixture all_services et l'objet ResponsesMock et les fixtures qui vont avec on ne peut pas faire ça comme ça.
Mis à jour par Nicolas Roche il y a plus d'un an
- Fichier 0003-toulouse-maelis-pass-get-direct-debit-order-paramete.patch 0003-toulouse-maelis-pass-get-direct-debit-order-paramete.patch ajouté
- Fichier 0002-toulouse-maelis-apply-direct-debit-order-WS-on-RL1-7.patch 0002-toulouse-maelis-apply-direct-debit-order-WS-on-RL1-7.patch ajouté
- Fichier 0001-toulouse-maelis-modify-regie-number-on-functest-7256.patch 0001-toulouse-maelis-modify-regie-number-on-functest-7256.patch ajouté
Nouveau rebase sur rebase sur #72638 :
Voilà, j'ai ajouté la possibilité d'imbriquer les context manager
Je me note ici le message de Benjamin qui m'a aidé à réécrire le patch 0002 (le seul qui a changé) :
tu peux instancier un requests_mock global et le partager entre test ResponsesSoap
@pytest.fixture def requests_mock(): return responses.RequestsMock() @pytest.fixture def family_service(requests_mock): yield ResponsesSoap(..., requests_mock=requests_mock)
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit bed1d0533d8688190647e381cff677fa528ffe78 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Sun Dec 18 23:22:52 2022 +0100 toulouse-maelis: pass get-direct-debit order parameters in QS (#72563) commit 70406863a4e9a31061b568a15d384f015bbab585 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Sun Dec 18 23:20:46 2022 +0100 toulouse-maelis: apply direct debit order WS on RL1 (#72563) commit b743d5db12f9001800fce06df42d00542f4c9048 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Sun Dec 18 23:46:21 2022 +0100 toulouse-maelis: modify regie number on functest (#72563)
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: modify regie number on functest (#72563)