toulouse-maelis: accéder aux données nécessaires pour l'inscription extra-scolaire et loisir (#73648) #59

Merged
nroche merged 1 commits from wip/73648-parsifal-get-person-unit-info into main 2023-02-02 11:15:33 +01:00
Owner

Nouveau endpoint pour accéder au WS getPersonUnitInfo.

Nouveau endpoint pour accéder au WS getPersonUnitInfo.
fpeters changed title from tolulouse-maelis: accéder aux données nécessaires pour l'inscription extra-scolaire et loisir (#73648) to toulouse-maelis: accéder aux données nécessaires pour l'inscription extra-scolaire et loisir (#73648) 2023-01-25 09:05:26 +01:00
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 2a43dc5db2 to 6a5b1443d6 2023-01-30 12:13:26 +01:00 Compare
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 6a5b1443d6 to 740ca7bfa0 2023-01-30 12:40:36 +01:00 Compare
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 740ca7bfa0 to fe647837dc 2023-01-30 12:41:03 +01:00 Compare
Author
Owner

(rebase)

(rebase)
nroche force-pushed wip/73648-parsifal-get-person-unit-info from fe647837dc to 182740c37e 2023-01-31 19:37:56 +01:00 Compare
Author
Owner

(rebase)

(rebase)
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 182740c37e to 0c0802568d 2023-02-01 12:35:27 +01:00 Compare
Author
Owner

(rebase)

(rebase)
Owner

(rebase)

Pour info il y a déjà une notification quand la branche est poussée; ça ne me semble pas utile de les doubler.

> (rebase) Pour info il y a déjà une notification quand la branche est poussée; ça ne me semble pas utile de les doubler.
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 0c0802568d to e98331cb9a 2023-02-01 14:20:33 +01:00 Compare
nroche force-pushed wip/73648-parsifal-get-person-unit-info from e98331cb9a to eb877622c0 2023-02-01 15:24:12 +01:00 Compare
fpeters requested changes 2023-02-01 16:00:11 +01:00
@ -2419,0 +2438,4 @@
perm='can_access',
parameters={
'person_id': {'description': "Numéro du responsale légal ou de l'enfant"},
'activity_id': {'description': "Numéro de l'activités"},
Owner

activité, singulier.

activité, singulier.
fpeters marked this conversation as resolved
@ -2419,0 +2470,4 @@
activity_id,
unit_id,
place_id,
ref_date=ref_date and ref_date.strftime(utils.json_date_format),
Owner

Il me semblerait plus opportun que l'adaptation au format soit effectuée dans la méthode appelée, plutôt qu'avoir à être répétée dans tous les appelants.

Il me semblerait plus opportun que l'adaptation au format soit effectuée dans la méthode appelée, plutôt qu'avoir à être répétée dans tous les appelants.
Owner

Il ne me semble pas que ça ait été compris; ci-dessus je notais que la gestion u format du paramètre ref_date aurait à gagner à être géré dans la méthode appelée (get_person_unit_info_raw). Dans la branche actualisée ça n'est toujours pas le cas, l'appelant (get_person_subscribing_info) contient toujours un code d'interprétation de ref_date (curieusement encore plus long qu'avant).

Il ne me semble pas que ça ait été compris; ci-dessus je notais que la gestion u format du paramètre ref_date aurait à gagner à être géré dans la méthode appelée (get_person_unit_info_raw). Dans la branche actualisée ça n'est toujours pas le cas, l'appelant (get_person_subscribing_info) contient toujours un code d'interprétation de ref_date (curieusement encore plus long qu'avant).
Author
Owner

Oui, j'ai anticipé la factorisation de l'appel au WS maélis, ce qui n'était pas une bonne idée. Il n'y a plus d'appelé.
(Concernant la taille du code d'interprétation de ref_date, voir la réponse au commentaire ci-dessous.)

Oui, j'ai anticipé la factorisation de l'appel au WS maélis, ce qui n'était pas une bonne idée. Il n'y a plus d'appelé. (Concernant la taille du code d'interprétation de ref_date, voir la réponse au commentaire ci-dessous.)
@ -5285,0 +5379,4 @@
resp = app.get(url, params=params, status=400)
assert resp.json['err'] == 1
assert resp.json['err_desc'] == 'invalid value for parameter "ref_date (YYYY-MM-DD expected)"'
Owner

Il est bien plus facile depuis w.c.s. de passer un paramètre vide que de ne pas passer de paramètre, pourquoi ce comportement contraire ?

Il est bien plus facile depuis w.c.s. de passer un paramètre vide que de ne pas passer de paramètre, pourquoi ce comportement contraire ?
Author
Owner

C'était pour utiliser le type date sur ce paramètre pour ne pas à avoir à gérer les cas d'erreur, mais c'était une mauvaise idée, parce ce type ne fonctionne pas avec un paramètre optionnel.

C'était pour utiliser le type date sur ce paramètre pour ne pas à avoir à gérer les cas d'erreur, mais c'était une mauvaise idée, parce ce type ne fonctionne pas avec un paramètre optionnel.
Author
Owner

C'est #72641

C'est #72641
Owner

toulouse-maelis: new endpoint to get subscribing info (#73648)

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

> toulouse-maelis: new endpoint to get subscribing info (#73648) Ce n'est pas le format attendu pour les messages de commit; ça devrait être "... add new ...".
nroche force-pushed wip/73648-parsifal-get-person-unit-info from eb877622c0 to 23040d4d37 2023-02-01 16:55:34 +01:00 Compare
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 23040d4d37 to 907141d026 2023-02-01 16:56:17 +01:00 Compare
Author
Owner

Remarques prises en compte.

Remarques prises en compte.
nroche requested review from fpeters 2023-02-01 19:28:12 +01:00
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 907141d026 to 7501fdd28d 2023-02-01 21:04:28 +01:00 Compare
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 7501fdd28d to 4aa1291e1f 2023-02-01 21:32:48 +01:00 Compare
Owner

Mes 50 cents: s/subscribing/subscription/

Mes 50 cents: s/subscribing/subscription/
nroche force-pushed wip/73648-parsifal-get-person-unit-info from 4aa1291e1f to bc32bb2f54 2023-02-02 10:33:45 +01:00 Compare
Owner

Le mot français "inscription" (un sujet) donne en anglais "subscription" (noun).

Le mot français "inscription" (un sujet) donne en anglais "subscription" (noun).
smihai approved these changes 2023-02-02 10:52:58 +01:00
@ -499,0 +508,4 @@
'dateRef': ref_date,
}
response = self.call('Activity', 'getPersonUnitInfo', getPersonUnitInfoRequestBean=params)
data = serialize_object(response)
Owner

J'aurais fait directement:

return serialize_object(response)

J'aurais fait directement: `return serialize_object(response)`
Author
Owner

oui, c'est #74084.
(tu as vraiment écris ce commentaire il y a une semaine ?)
edit: d'après le mail reçu, non : Date: Thu, 02 Feb 2023 10:52:58 +0100)

oui, c'est #74084. (tu as vraiment écris ce commentaire il y a une semaine ?) edit: d'après le mail reçu, non : Date: Thu, 02 Feb 2023 10:52:58 +0100)
nroche force-pushed wip/73648-parsifal-get-person-unit-info from bc32bb2f54 to 76d8e2e7be 2023-02-02 10:58:27 +01:00 Compare
nroche merged commit 76d8e2e7be into main 2023-02-02 11:15:33 +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#59
No description provided.