Development #53884
Caluire Axel: endpoints pour récupérer les factures
0%
Description
Visualiser les factures, avec leur état de règlement (web service : GetFacturesaPayer / GetListFactures / GetPdfFacture).
Fichiers
Révisions associées
caluire-axel: add invoices_history endpoint (#53884)
caluire-axel: add invoice endpoint (#53884)
caluire-axel: add invoice_pdf endpoint (#53884)
Historique
Mis à jour par Nicolas Roche il y a presque 3 ans
- Fichier 0004-caluire-axel-add-invoice_pdf-endpoint-53884.patch 0004-caluire-axel-add-invoice_pdf-endpoint-53884.patch ajouté
- Fichier 0003-caluire-axel-add-invoice-endpoint-53884.patch 0003-caluire-axel-add-invoice-endpoint-53884.patch ajouté
- Fichier 0002-caluire-axel-add-invoices_history-endpoint-53884.patch 0002-caluire-axel-add-invoices_history-endpoint-53884.patch ajouté
- Fichier 0001-caluire-axel-add-invoices-endpoint-53884.patch 0001-caluire-axel-add-invoices-endpoint-53884.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Permier jet, en attendant d'avoir des factures pour tester/adapter.
D'après la documentation, Le WS GetListFactures attend un paramètre obligatoire NbMois pour le Nombre de mois d'historique à renvoyer.J'ai propagé ce paramètre jusqu'aux endpoints et donné une valeur par défaut :
- 3 mois pour la récupération des factures payées
- 12 mois pour la recherche d'une facture précise
Aussi, en cas de mauvais paramètre je lève une erreur 400 (comme pour les mauvaises dates) et lorsque l'on est dans le contexte du endpoint invoice_pdf, je lève une 404 (comme c'était déjà prévu avant).
Mis à jour par Lauréline Guérin il y a presque 3 ans
Petit point de détail: dans un cas tu as nb_mounts_limit='3'
, dans l'autre nb_mounts_limit='12'
; possible d'harmoniser ?
Ca te va si on attend d'avoir des données de test pour valider ?
Mis à jour par Nicolas Roche il y a presque 3 ans
Oui, j'attends les données pour tester puis passer le ticket à "solution proposée".
Pour la durée de recherche par défaut il va falloir que tu tranches.
Mis à jour par Nicolas Roche il y a presque 3 ans
Branche rebasée, mais il manque encore les factures PDF pour tester 0004.
Mis à jour par Nicolas Roche il y a presque 3 ans
- Fichier 0004-caluire-axel-add-invoice_pdf-endpoint-53884.patch 0004-caluire-axel-add-invoice_pdf-endpoint-53884.patch ajouté
- Fichier 0003-caluire-axel-add-invoice-endpoint-53884.patch 0003-caluire-axel-add-invoice-endpoint-53884.patch ajouté
- Fichier 0002-caluire-axel-add-invoices_history-endpoint-53884.patch 0002-caluire-axel-add-invoices_history-endpoint-53884.patch ajouté
- Fichier 0001-caluire-axel-add-invoices-endpoint-53884.patch 0001-caluire-axel-add-invoices-endpoint-53884.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Nicolas Roche il y a presque 3 ans
En fait ça m'irait de passer ces patchs indépendants puis de les corriger ensuite.
(pour ne plus avoir à les rebaser)
Mis à jour par Lauréline Guérin il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Ok, peux-tu harmoniser le default de nb_mounts_limit ?
Je vote pour 12 mois
Mis à jour par Nicolas Roche il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 0e710613c08a11e92f08465a745c791d4d3cf2cf Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Tue May 11 18:07:29 2021 +0200 caluire-axel: add invoice_pdf endpoint (#53884) commit 4884f6d82c6f38bf895a5e21dc1e6b0b6a9f7ff0 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Tue May 11 17:20:26 2021 +0200 caluire-axel: add invoice endpoint (#53884) commit 5f50aaba77714ebcc958529d6b0da28ea07f8bd1 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Tue May 11 15:08:27 2021 +0200 caluire-axel: add invoices_history endpoint (#53884) commit a07068d1294c62097093f8617094e5166153dd62 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Mon May 10 15:38:19 2021 +0200 caluire-axel: add invoices endpoint (#53884)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
caluire-axel: add invoices endpoint (#53884)