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

toulouse-maelis: ajouter des endpoints pour valider le panier (#73961) #65

Merged
nroche merged 5 commits from wip/73961-parsifal-basket into main 2023-02-02 12:09:57 +01:00
Conversation 2 Commits 5 Files Changed 8 +531
nroche commented 2023-01-30 03:45:33 +01:00
Owner
Copy Link

Le panier est utilisé par maélis pour permettre de créer une unique facture par régie, pour de multiples inscriptions.

Le panier est utilisé par maélis pour permettre de créer une unique facture par régie, pour de multiples inscriptions.
nroche added 10 commits 2023-01-30 03:45:34 +01:00
gitea-wip/passerelle/pipeline/pr-main There was a failure building this commit Details
172c895e05 toulouse-maelis: add endpoint to validate basket (#73961)
nroche changed title from (wip) toulouse-maelis: ajouter des endpoints pour valider le panier (#73961) to WIP: toulouse-maelis: ajouter des endpoints pour valider le panier (#73961) 2023-01-30 03:49:00 +01:00
nroche force-pushed wip/73961-parsifal-basket from 172c895e05 to 66b76b5291 2023-01-31 08:37:05 +01:00 Compare
nroche changed title from WIP: toulouse-maelis: ajouter des endpoints pour valider le panier (#73961) to toulouse-maelis: ajouter des endpoints pour valider le panier (#73961) 2023-01-31 19:41:06 +01:00
fpeters commented 2023-02-01 16:04:12 +01:00
Owner
Copy Link

Il y a profusion de commits sans rapport dans la branche, ça empêche la relecture.

Il y a profusion de commits sans rapport dans la branche, ça empêche la relecture.
nroche force-pushed wip/73961-parsifal-basket from ab9b50764d to 1abe057607 2023-02-01 17:36:31 +01:00 Compare
nroche commented 2023-02-01 17:42:39 +01:00
Author
Owner
Copy Link

J'ai retiré les commmits (j'en ai eu besoin pour tester le panier "en vrai").
(et aussi, désolé d'avoir collé ces 5 endpoints dans la même PR)

J'ai retiré les commmits (j'en ai eu besoin pour tester le panier "en vrai"). (et aussi, désolé d'avoir collé ces 5 endpoints dans la même PR)
fpeters requested changes 2023-02-01 17:59:26 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2418,1 +2429,4 @@
@endpoint(
display_category='Inscriptions',
description="Lecture du pannier de la famille",
fpeters commented 2023-02-01 17:46:31 +01:00
Owner
Copy Link

panier. (ici et ailleurs).

panier. (ici et ailleurs).
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2419,0 +2458,4 @@
basket = self.get_basket_raw(family_id)
if not basket or basket.get('id') != basket_id:
raise APIError("no '%s' basket on '%s' family" % (basket_id, family_id), err_code='not-found')
fpeters commented 2023-02-01 17:51:58 +01:00
Owner
Copy Link

Si je comprends bien il y a un seul panier par famille, pourquoi alors imposer de passer un basket_id ici ?

Si je comprends bien il y a un seul panier par famille, pourquoi alors imposer de passer un basket_id ici ?
nroche commented 2023-02-01 19:26:11 +01:00
Author
Owner
Copy Link

Pour des fausses raisons de sécurité.
J'ai retiré ce paramètre.

Pour des fausses raisons de sécurité. J'ai retiré ce paramètre.
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2419,0 +2480,4 @@
basket = self.get_basket_raw(family_id)
if not basket or basket.get('id') != basket_id:
raise APIError("no '%s' basket on '%s' family" % (basket_id, family_id), err_code='not-found')
for line in basket['lignes']:
fpeters commented 2023-02-01 17:52:16 +01:00
Owner
Copy Link

(même question)

(même question)
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2419,0 +2512,4 @@
basket = self.get_basket_raw(family_id)
if basket['id'] != basket_id:
raise APIError("no '%s' basket on '%s' family" % (basket_id, family_id), err_code='not-found')
fpeters commented 2023-02-01 17:53:39 +01:00
Owner
Copy Link

(ici aussi)

(ici aussi)
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -2419,0 +2539,4 @@
family_id = family_id or self.get_link(NameID).family_id
basket = self.get_basket_raw(family_id)
if not basket or basket.get('id') != basket_id:
raise APIError("no '%s' basket on '%s' family" % (basket_id, family_id), err_code='not-found')
fpeters commented 2023-02-01 17:53:54 +01:00
Owner
Copy Link

(et ici)

(et ici)
nroche force-pushed wip/73961-parsifal-basket from 1abe057607 to c78621f56b 2023-02-01 19:24:59 +01:00 Compare
nroche force-pushed wip/73961-parsifal-basket from c78621f56b to 62e835cc3a 2023-02-01 21:41:10 +01:00 Compare
smihai approved these changes 2023-02-02 11:14:41 +01:00
nroche force-pushed wip/73961-parsifal-basket from 62e835cc3a to e9e9aec674 2023-02-02 11:51:28 +01:00 Compare
nroche merged commit e9e9aec674 into main 2023-02-02 12:09:57 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
smihai
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#65
No description provided.
Delete Branch "wip/73961-parsifal-basket"

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?