entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 14 Releases Activity

toulouse-maelis: correct for-payment parameter usage (#77110) #232

Merged
nroche merged 1 commits from wip/77110-parsifal-correct-for-payment into main 2023-11-10 10:53:45 +01:00
Conversation 0 Commits 1 Files Changed 2 +51 -15
nroche commented 2023-04-28 12:13:08 +02:00
Owner
Copy Link

Les scénarios deviennent :

  • On s'inscrit (ras)

  • à T on valide le panier
    => basket_generation_date = T

  • Soit on ne paye pas (pas d'appel avec '?for-payment') :
    => à T+cancel_invoice_delay, la facture n'est plus affichée (mais reste payable).
    => à T+cancel_invoice_delay+max_payment_delay, la facture est annulée

  • Soit à T' (T' < T+cancel_invoice_delay) on paye :
    => start_payment_date = T'
    => on ne peut plus (re-)payer la facture ensuite : on n'a le droit qu'à un seul clic sur 'Voir et payer'.
    => à T'+max_payment_delay, la facture est annulée.

J'ai repris les tests qui ne ne prenait pas en compte ce tout dernier point.

Les scénarios deviennent : * On s'inscrit (ras) * à T on valide le panier => basket_generation_date = T * Soit on ne paye pas (pas d'appel avec '?for-payment') : => à T+cancel_invoice_delay, la facture n'est plus affichée (mais reste payable). => à T+cancel_invoice_delay+max_payment_delay, la facture est annulée * Soit à T' (T' < T+cancel_invoice_delay) on paye : => start_payment_date = T' => on ne peut plus (re-)payer la facture ensuite : on n'a le droit qu'à un seul clic sur 'Voir et payer'. => à T'+max_payment_delay, la facture est annulée. J'ai repris les tests qui ne ne prenait pas en compte ce tout dernier point.
nroche force-pushed wip/77110-parsifal-correct-for-payment from 697e6bd056 to cf6da3e4a6 2023-05-10 17:50:24 +02:00 Compare
nroche reviewed 2023-05-16 12:37:54 +02:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -4023,3 +4023,3 @@
if i.status() in ['paid', 'notified']
]
return {'data': invoices}
return {'has_invoice_for_payment': True, 'data': invoices}
nroche commented 2023-05-16 12:37:54 +02:00
Author
Owner
Copy Link

Je laisse le patch en l'état : il renvoie 'has_invoice_for_payment': True
également sur l'historique des factures (alors que ce n'est plus requis par #76853),
par prudence, parce que je n'ai plus de quoi tester actuellement le scénario d'inscription sur l'environnement Maélis de DEV.

Je laisse le patch en l'état : il renvoie 'has_invoice_for_payment': True également sur l'historique des factures (alors que ce n'est plus requis par #76853), par prudence, parce que je n'ai plus de quoi tester actuellement le scénario d'inscription sur l'environnement Maélis de DEV.
nroche force-pushed wip/77110-parsifal-correct-for-payment from cf6da3e4a6 to 0f36d4c13b 2023-05-16 12:38:24 +02:00 Compare
bdauvergne approved these changes 2023-11-09 19:38:25 +01:00
bdauvergne left a comment
Owner
Copy Link

À rebaser.

À rebaser.
nroche force-pushed wip/77110-parsifal-correct-for-payment from 0f36d4c13b to 979e531b3a 2023-11-10 10:41:38 +01:00 Compare
nroche merged commit 979e531b3a into main 2023-11-10 10:53:45 +01:00
nroche deleted branch wip/77110-parsifal-correct-for-payment 2023-11-10 10:53:45 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
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/passerelle#232
No description provided.
Delete Branch "wip/77110-parsifal-correct-for-payment"

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?