toulouse-maelis: endpoint d'inscription extra-scolaire et loisir (#73649) #60

Merged
nroche merged 1 commits from wip/73649-parsifal-add-person-unit-basket into main 2023-02-02 11:45:35 +01:00
Owner

Ajout d'un endpoint pour faire une inscription via le panier.
Il faut passer par le panier parce que dans la même démarche on proposera un agenda pour les réservations.

Il restera à préciser la semaine type lors de l'inscription.
(pas encore réalisable en l'état : pas de tests côté Sigec, pas les bonnes données en base pour tester...)
cf https://redmine.sigec.fr/issues/1565

Ajout d'un endpoint pour faire une inscription via le panier. Il faut passer par le panier parce que dans la même démarche on proposera un agenda pour les réservations. Il restera à préciser la semaine type lors de l'inscription. (pas encore réalisable en l'état : pas de tests côté Sigec, pas les bonnes données en base pour tester...) cf https://redmine.sigec.fr/issues/1565
fpeters changed title from tolulouse-maelis: endpoint d'inscription extra-scolaire et loisir (#73649) to toulouse-maelis: endpoint d'inscription extra-scolaire et loisir (#73649) 2023-01-25 09:11:56 +01:00
nroche force-pushed wip/73649-parsifal-add-person-unit-basket from 425785afdb to bf6823964d 2023-02-01 15:42:17 +01:00 Compare
Owner

toulouse-maelis: new endpoint to subscribe into basket (#73649)

Ce n'est pas le format attendu pour les messages de commit; ça devrait être "... add new ...".

> toulouse-maelis: new endpoint to subscribe into basket (#73649) Ce n'est pas le format attendu pour les messages de commit; ça devrait être "... add new ...".
fpeters reviewed 2023-02-01 16:10:53 +01:00
@ -5285,0 +5375,4 @@
'controlResult': {
'controlOK': False,
'message': 'E1019 : Il existe déjà une inscription à cette unité',
},
Owner

Pourquoi err: 0 alors qu'on est sur une erreur ?

Pourquoi err: 0 alors qu'on est sur une erreur ?
Author
Owner

C'était pour reproduire ce qui est proposé sur https://dev.entrouvert.org/issues/73648.
Mais ce n'était pas une bonne idée, parce qu'ici on a aucune information supplémentaire à renvoyer.
Aussi, ici le endpoint sera appelé dans les workflows (alors que dans le cas précédent il sera appelé dans les formulaires).
Oui, il faut sortir en erreur si l'inscription n'a pas eu lieu.

C'était pour reproduire ce qui est proposé sur https://dev.entrouvert.org/issues/73648. Mais ce n'était pas une bonne idée, parce qu'ici on a aucune information supplémentaire à renvoyer. Aussi, ici le endpoint sera appelé dans les workflows (alors que dans le cas précédent il sera appelé dans les formulaires). Oui, il faut sortir en erreur si l'inscription n'a pas eu lieu.
nroche force-pushed wip/73649-parsifal-add-person-unit-basket from bf6823964d to ceddda9afe 2023-02-01 17:13:16 +01:00 Compare
fpeters reviewed 2023-02-01 17:16:01 +01:00
@ -5285,0 +5367,4 @@
}
resp = app.post_json(url + '?family_id=311323', params=params)
assert resp.json['err'] == 'control-fails'
assert resp.json['err_desc'] == 'E1019 : Il existe déjà une inscription à cette unité'
Owner

Il devrait être assez convenu que dans "err" doit se trouver un nombre.

Il devrait être assez convenu que dans "err" doit se trouver un nombre.
Author
Owner

C'était pour reproduire ce qui est proposé sur https://dev.entrouvert.org/issues/73648.

(commentaire déplacé ci-dessus)

> C'était pour reproduire ce qui est proposé sur https://dev.entrouvert.org/issues/73648. (commentaire déplacé ci-dessus)
Owner

C'était pour reproduire ce qui est proposé sur https://dev.entrouvert.org/issues/73648.

Essaie de répondre au niveau du point soulevé (en cliquant sur "répondre" dessous) plutôt que faire un commentaire global qui peut être difficile à réassocier.

> C'était pour reproduire ce qui est proposé sur https://dev.entrouvert.org/issues/73648. Essaie de répondre au niveau du point soulevé (en cliquant sur "répondre" dessous) plutôt que faire un commentaire global qui peut être difficile à réassocier.
nroche force-pushed wip/73649-parsifal-add-person-unit-basket from ceddda9afe to ead1adfc70 2023-02-01 17:49:20 +01:00 Compare
nroche force-pushed wip/73649-parsifal-add-person-unit-basket from ead1adfc70 to 1225141fe0 2023-02-01 21:37:11 +01:00 Compare
nroche force-pushed wip/73649-parsifal-add-person-unit-basket from 1225141fe0 to 7ed8761090 2023-02-02 11:22:38 +01:00 Compare
nroche force-pushed wip/73649-parsifal-add-person-unit-basket from 7ed8761090 to 189747ff55 2023-02-02 11:27:45 +01:00 Compare
smihai approved these changes 2023-02-02 11:42:33 +01:00
nroche merged commit 189747ff55 into main 2023-02-02 11:45:35 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#60
No description provided.