entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 10 Releases Wiki Activity

ajouter ?for-payment quand on lit une facture pour la payer (#76853) #85

Merged
bdauvergne merged 3 commits from wip/76853-lingo-lors-du-get-invoice-pour-p into main 2024-02-27 09:26:45 +01:00
Conversation 2 Commits 3 Files Changed 4 +84 -4
bdauvergne commented 2023-04-21 01:59:15 +02:00
Owner
Copy Link

Le paramètre supplémentaire n'étant pas supporté partout, il n'est ajouté que si la source a annoncé son support sur son endpoint /invoices/.

Le paramètre supplémentaire n'étant pas supporté partout, il n'est ajouté que si la source a annoncé son support sur son endpoint `/invoices/`.
bdauvergne changed title from WIP: ajouter ?for-payment quand on lit une facture pour la payer (#76853) to ajouter ?for-payment quand on lit une facture pour la payer (#76853) 2023-04-21 02:25:05 +02:00
nroche reviewed 2023-04-28 11:20:12 +02:00
combo/apps/lingo/models.py Outdated
@ -369,3 +379,4 @@
if not self.is_remote():
return self.basketitem_set.get(pk=invoice_id)
url = self.webservice_url + '/invoice/%s/' % invoice_id
if self.has_invoice_for_payment:
nroche commented 2023-04-28 11:20:12 +02:00
Owner
Copy Link

Ce paramètre est passé aux connecteurs également lorsque l'on demande à voir une facture historique.
Il faudra y penser quand on le prendra en charge dans les connecteurs.

Ce paramètre est passé aux connecteurs également lorsque l'on demande à voir une facture historique. Il faudra y penser quand on le prendra en charge dans les connecteurs.
bdauvergne commented 2023-05-02 07:52:46 +02:00
Author
Owner
Copy Link

J'avais ajouté un paramètre for_payment à get_invoice() pour justement gérer ce cas, j'ai oublié de le mettre dans la condition c'est corrigé. Le paramètre n'est envoyé que dans le cas PayView désormais.

J'avais ajouté un paramètre for_payment à get_invoice() pour justement gérer ce cas, j'ai oublié de le mettre dans la condition c'est corrigé. Le paramètre n'est envoyé que dans le cas PayView désormais.
nroche marked this conversation as resolved
nroche reviewed 2023-04-28 11:21:17 +02:00
combo/apps/lingo/models.py Outdated
@ -370,2 +380,4 @@
return self.basketitem_set.get(pk=invoice_id)
url = self.webservice_url + '/invoice/%s/' % invoice_id
if self.has_invoice_for_payment:
url += '?for-payment'
nroche commented 2023-04-28 11:21:17 +02:00
Owner
Copy Link

Le nom du paramètre (avec '-') ne permet pas d'avoir une variable dans
les paramètres du enpoint du connecteur, mais oblige d'utiliser le
kwargs.
Peut-être que c'est mieux ainsi : que l'on ne puisse pas documenter
dans la page du connecteur la prise en compte de la variable qui est à
usage interne de Publik, je ne sais pas.

Le nom du paramètre (avec '-') ne permet pas d'avoir une variable dans les paramètres du enpoint du connecteur, mais oblige d'utiliser le kwargs. Peut-être que c'est mieux ainsi : que l'on ne puisse pas documenter dans la page du connecteur la prise en compte de la variable qui est à usage interne de Publik, je ne sais pas.
bdauvergne commented 2023-05-02 07:53:40 +02:00
Author
Owner
Copy Link

Non t'as raison j'oublie toujours cette contrainte, je vais modifier ça si tu peux le prendre en compte coté maelis.

Non t'as raison j'oublie toujours cette contrainte, je vais modifier ça si tu peux le prendre en compte coté maelis.
nroche marked this conversation as resolved
nroche approved these changes 2023-04-28 11:27:00 +02:00
nroche left a comment
Owner
Copy Link

Dans lingo, je ne vois qu'un seul appel au endpoint '/invoice/%s/'.
Vu que le paramètre 'for-payment' est systématiquement passé lorsque le connecteur le prend en compte, je ne comprends pas sa plus value.
J'imagine que ça permettra d'adapter le comportement de lingo à l'avenir.

Ok, c'est plus le cas à présent.

~~Dans lingo, je ne vois qu'un seul appel au endpoint '/invoice/%s/'. Vu que le paramètre 'for-payment' est systématiquement passé lorsque le connecteur le prend en compte, je ne comprends pas sa plus value. J'imagine que ça permettra d'adapter le comportement de lingo à l'avenir.~~ Ok, c'est plus le cas à présent.
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 0cfd8eca32 to 2e7c820bcf 2023-05-02 07:52:04 +02:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 2e7c820bcf to e41f878141 2023-05-02 07:56:29 +02:00 Compare
bdauvergne requested review from nroche 2023-05-02 07:56:32 +02:00
bdauvergne removed review request for nroche 2023-05-02 08:30:15 +02:00
bdauvergne requested review from nroche 2023-05-02 08:30:55 +02:00
nroche commented 2023-05-03 15:25:15 +02:00
Owner
Copy Link

Benjamin, là je vais pas avoir le temps de re-tester (via #77110) avant le freeze.
Comme j'ai déjà validé, si c'est pressé pousses en l'état et sinon je reprends #77110 dès que je peux.

Benjamin, là je vais pas avoir le temps de re-tester (via #77110) avant le freeze. Comme j'ai déjà validé, si c'est pressé pousses en l'état et sinon je reprends #77110 dès que je peux.
bdauvergne commented 2023-05-03 17:00:27 +02:00
Author
Owner
Copy Link

Pour moi il n'y a rien qui presse, c'est toi qui décide.

Pour moi il n'y a rien qui presse, c'est toi qui décide.
nroche approved these changes 2023-05-10 17:50:43 +02:00
nroche left a comment
Owner
Copy Link

Ack.
J'ai proposé une éventuelle amélioration, mais indépendamment j'ai mis à jour entrouvert/passerelle#232
et testé que tout fonctionne bien en l'état.

Ack. J'ai proposé une éventuelle amélioration, mais indépendamment j'ai mis à jour https://git.entrouvert.org/entrouvert/passerelle/pulls/232 et testé que tout fonctionne bien en l'état.
combo/apps/lingo/models.py Outdated
@ -355,2 +358,4 @@
if items.get('err'):
raise RegieException(regie_exc_msg)
has_invoice_for_payment = items.get('has_invoice_for_payment', False)
if self.has_invoice_for_payment != has_invoice_for_payment:
nroche commented 2023-05-10 17:42:01 +02:00
Owner
Copy Link

Ici on pourrait également conditionner sur not history,
afin de ne pas à avoir à répondre {'has_invoice_for_payment': True}
dans le endpoint des connecteurs qui renvoient l'historique des factures
(ou pas, c'est juste que ça me semblerait plus robuste).

Ici on pourrait également conditionner sur `not history`, afin de ne pas à avoir à répondre `{'has_invoice_for_payment': True}` dans le endpoint des connecteurs qui renvoient l'historique des factures (ou pas, c'est juste que ça me semblerait plus robuste).
nroche marked this conversation as resolved
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from e41f878141 to 0927b18230 2023-05-15 10:53:49 +02:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 0927b18230 to 08abf1fc56 2023-05-15 11:01:51 +02:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 08abf1fc56 to df8ec5c279 2023-05-15 12:40:17 +02:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from df8ec5c279 to 74ae8c1e97 2024-02-24 17:06:52 +01:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 74ae8c1e97 to 48ceec92b5 2024-02-24 17:10:19 +01:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 48ceec92b5 to 67f9c99c00 2024-02-24 17:14:08 +01:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 67f9c99c00 to 742edf3046 2024-02-24 17:28:09 +01:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 742edf3046 to 335c7b027a 2024-02-24 17:36:49 +01:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 335c7b027a to 91f0eb19f1 2024-02-26 19:28:17 +01:00 Compare
bdauvergne force-pushed wip/76853-lingo-lors-du-get-invoice-pour-p from 91f0eb19f1 to bbb12f507f 2024-02-26 19:28:37 +01:00 Compare
bdauvergne merged commit bbb12f507f into main 2024-02-27 09:26:45 +01:00
bdauvergne deleted branch wip/76853-lingo-lors-du-get-invoice-pour-p 2024-02-27 09:26:45 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/combo#85
No description provided.
Delete Branch "wip/76853-lingo-lors-du-get-invoice-pour-p"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?