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

toulouse-maelis: réservations extra-scolaire (#75144) #136

Merged
nroche merged 4 commits from wip/75144-parsifal-agenda-on-subscription into main 2023-03-24 08:49:55 +01:00
Conversation 1 Commits 4 Files Changed 11 +2787 -3
nroche commented 2023-03-11 20:02:15 +01:00
Owner
Copy Link

(cours)

(cours)
nroche force-pushed wip/75144-parsifal-agenda-on-subscription from 2149459015 to df76a02a07 2023-03-19 10:36:38 +01:00 Compare
nroche changed title from WIP: toulouse-maelis: réservations loisir et extra-scolaire (#75144) to WIP: toulouse-maelis: réservations extra-scolaire (#75144) 2023-03-19 10:36:48 +01:00
nroche commented 2023-03-19 10:42:15 +01:00
Author
Owner
Copy Link

Branche WIP parce que contient 4 commits que je vais faire passer un par un,
mais bon, qui sait, sur un ack de complaisance ça pourrait passer d'un coup.
(je dis ça parce que Stéphane est pressé d'avoir ce dév).

Dans le ticket redmine j'ai posé des le déroulé et des capture d'écrans
(et aussi ce qui serait peut-être en trop dans le code).
J'ai posé ma démarche de test ici :
https://demarches-parsifal.test.entrouvert.org/backoffice/workflows/140/

Branche WIP parce que contient 4 commits que je vais faire passer un par un, mais bon, qui sait, sur un ack de complaisance ça pourrait passer d'un coup. (je dis ça parce que Stéphane est pressé d'avoir ce dév). Dans le ticket redmine j'ai posé des le déroulé et des capture d'écrans (et aussi ce qui serait peut-être en trop dans le code). J'ai posé ma démarche de test ici : https://demarches-parsifal.test.entrouvert.org/backoffice/workflows/140/
nroche force-pushed wip/75144-parsifal-agenda-on-subscription from df76a02a07 to fabe825f15 2023-03-19 13:41:54 +01:00 Compare
nroche force-pushed wip/75144-parsifal-agenda-on-subscription from fabe825f15 to 92a19d123d 2023-03-20 16:40:42 +01:00 Compare
nroche force-pushed wip/75144-parsifal-agenda-on-subscription from 92a19d123d to 47f12cd12f 2023-03-20 17:11:16 +01:00 Compare
nroche changed title from WIP: toulouse-maelis: réservations extra-scolaire (#75144) to toulouse-maelis: réservations extra-scolaire (#75144) 2023-03-20 17:23:40 +01:00
lguerin reviewed 2023-03-20 21:20:18 +01:00
passerelle/contrib/toulouse_maelis/models.py
@ -2844,0 +2896,4 @@
'idAct': activity_id,
}
}
response = self.call('Activity', 'getPersonScheduleList', **payload)
lguerin commented 2023-03-20 21:18:46 +01:00
Owner
Copy Link

est-ce qu'il y aurait moyen de factoriser avec ce qui est fait dans la méthode get_bookings ? Avoir un get_booking_for_activity ?

est-ce qu'il y aurait moyen de factoriser avec ce qui est fait dans la méthode `get_bookings` ? Avoir un `get_booking_for_activity` ?
nroche commented 2023-03-21 14:17:15 +01:00
Author
Owner
Copy Link

Non, pas moyen, la configuration de maélis diffère suivant le périmètre péri-scolaie d'un côté ou loisirs/extra-scolaire de l'autre.
Ici on traite une seule activité, alors que pour le périscolaire on en traite plusieurs.
Par exemple ça diffère sur l'affichage dans le widget :
booking['details']['activity_id'] = unit['unit']['idUnit']
et aussi sur les libellés à afficher (dans le péri-sco on affiche la nature).
(là le code va devenir illisible)

Non, pas moyen, la configuration de maélis diffère suivant le périmètre péri-scolaie d'un côté ou loisirs/extra-scolaire de l'autre. Ici on traite une seule activité, alors que pour le périscolaire on en traite plusieurs. Par exemple ça diffère sur l'affichage dans le widget : ` booking['details']['activity_id'] = unit['unit']['idUnit']` et aussi sur les libellés à afficher (dans le péri-sco on affiche la nature). (là le code va devenir illisible)
passerelle/contrib/toulouse_maelis/models.py
@ -2844,0 +3016,4 @@
available_bookings = [b['id'] for b in bookings if b['disabled'] is False]
updated = {}
bookings_to_update = []
lguerin commented 2023-03-20 21:19:44 +01:00
Owner
Copy Link

pareil, possibilité de factoriser avec ce qui est fait dans le endpoint update_child_agenda ?

pareil, possibilité de factoriser avec ce qui est fait dans le endpoint `update_child_agenda` ?
nroche commented 2023-03-21 14:29:49 +01:00
Author
Owner
Copy Link

Non, il s'agit de 2 WS maélis différents, et la structure et le nom des champs ne sont pas les même d'un WS à l'autre.
ex: activity['idAct'] vs activity['idActivity']
Surtout, rien ne garanti que les WS ne vont pas évoluer indépendamment (parce que c'est le cas ailleurs).
Après oui, j'aimerais bien aussi factoriser le code, mais je pense qu'il vaut mieux ne pas le faire avant d'avoir passer l'étape de la recette par les agents à Toulouse.

Non, il s'agit de 2 WS maélis différents, et la structure et le nom des champs ne sont pas les même d'un WS à l'autre. ex: activity['idAct'] vs activity['idActivity'] Surtout, rien ne garanti que les WS ne vont pas évoluer indépendamment (parce que c'est le cas ailleurs). Après oui, j'aimerais bien aussi factoriser le code, mais je pense qu'il vaut mieux ne pas le faire avant d'avoir passer l'étape de la recette par les agents à Toulouse.
lguerin approved these changes 2023-03-21 14:19:23 +01:00
nroche force-pushed wip/75144-parsifal-agenda-on-subscription from 47f12cd12f to 2e85f7e013 2023-03-24 08:38:13 +01:00 Compare
nroche merged commit 2e85f7e013 into main 2023-03-24 08:49:55 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
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#136
No description provided.
Delete Branch "wip/75144-parsifal-agenda-on-subscription"

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?