Projet

Général

Profil

Bug #72563

toulouse-maelis: prévoir d'appeler le WS get-direct-debit-order depuis combo

Ajouté par Nicolas Roche il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
16 décembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Lié à Passerelle - Development #72524: toulouse-maelis: intégrer la livraison du 14/12/2022Fermé16 décembre 2022

Actions
Lié à Passerelle - Development #72288: toulouse-maelis: (Faturation) endpoints pour lire écrire les coordonnées bancairesFermé12 décembre 2022

Actions

Révisions associées

Révision b743d5db (diff)
Ajouté par Nicolas Roche il y a plus d'un an

toulouse-maelis: modify regie number on functest (#72563)

Révision 70406863 (diff)
Ajouté par Nicolas Roche il y a plus d'un an

toulouse-maelis: apply direct debit order WS on RL1 (#72563)

Révision bed1d053 (diff)
Ajouté par Nicolas Roche il y a plus d'un an

toulouse-maelis: pass get-direct-debit order parameters in QS (#72563)

Historique

#1

Mis à jour par Nicolas Roche il y a plus d'un an

  • Description mis à jour (diff)
#2

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

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

Mis à jour par Nicolas Roche il y a plus d'un an

  • Assigné à mis à Nicolas Roche
#5

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 :

#6

Mis à jour par Nicolas Roche il y a plus d'un an

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.

#7

Mis à jour par Benjamin Dauvergne il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée
#8

Mis à jour par Robot Gitea il y a plus d'un an

  • Statut changé de Solution validée à Solution proposée
#9

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 :

Ça me parait bien. À pousser vendredi.

#10

Mis à jour par Nicolas Roche il y a plus d'un an

J'ajoute un dernier patch qui sera requis après rebase sur #72638 qui améliore le nouveau code de test SOAP.

#11

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)

#12

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.

#13

Mis à jour par Nicolas Roche il y a plus d'un an

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)
#14

Mis à jour par Benjamin Dauvergne il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée
#15

Mis à jour par Nicolas Roche il y a plus d'un an

(branche re-basée sur main)

#16

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)
#17

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

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

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF