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

toulouse-maelis: booking endpoints (#72774) #19

Merged
lguerin merged 2 commits from wip/72774-toulouse-maelis-bookings into main 2023-01-05 16:37:14 +01:00
Conversation 1 Commits 2 Files Changed 7 +1172 -1
lguerin commented 2023-01-03 16:43:48 +01:00
Owner
Copy Link
No description provided.
lguerin changed title from WIP toulouse-maelis: booking endpoints (#72774) to WIP: toulouse-maelis: booking endpoints (#72774) 2023-01-03 16:43:58 +01:00
lguerin force-pushed wip/72774-toulouse-maelis-bookings from 36f3f54caf to 5a222eff65 2023-01-05 10:22:00 +01:00 Compare
lguerin force-pushed wip/72774-toulouse-maelis-bookings from 5a222eff65 to 3c4c9117a6 2023-01-05 15:03:38 +01:00 Compare
lguerin force-pushed wip/72774-toulouse-maelis-bookings from 3c4c9117a6 to 1b8c92a35f 2023-01-05 15:21:44 +01:00 Compare
lguerin changed title from WIP: toulouse-maelis: booking endpoints (#72774) to toulouse-maelis: booking endpoints (#72774) 2023-01-05 15:23:35 +01:00
lguerin reviewed 2023-01-05 15:24:43 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -1616,0 +1686,4 @@
bookings.append(booking)
# sort bookings
activity_types = ['ACCMAT', 'RESTSCOL']
lguerin commented 2023-01-05 15:24:43 +01:00
Author
Owner
Copy Link

Pour le moment, les seuls types d'activités que j'ai pu manipuler avec les données que j'ai. Au pire, si on tombe sur un nouveau type, le tri sera un peu moisi, mais ça sera juste un pb d'affichage.

Pour le moment, les seuls types d'activités que j'ai pu manipuler avec les données que j'ai. Au pire, si on tombe sur un nouveau type, le tri sera un peu moisi, mais ça sera juste un pb d'affichage.
lguerin reviewed 2023-01-05 15:25:30 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -1616,0 +1657,4 @@
for result_data in result:
for schedule in result_data['activityScheduleList']:
activity = schedule['activity']
if activity['activityType']['natureSpec']['code'] not in ['A', 'R']:
lguerin commented 2023-01-05 15:25:30 +01:00
Author
Owner
Copy Link

Les seuls codes que j'ai pu manipuler avec les données que j'ai. Si on en a d'autres, les activités associées seront ignorées, il faudra mettre cette liste à jour.

Les seuls codes que j'ai pu manipuler avec les données que j'ai. Si on en a d'autres, les activités associées seront ignorées, il faudra mettre cette liste à jour.
lguerin commented 2023-01-05 15:27:36 +01:00
Author
Owner
Copy Link

Note: c'est lent (une requête par mois pour récupérer les agendas), il faudrait peut-être envisager d'ajouter un peu de cache ?

Note: c'est lent (une requête par mois pour récupérer les agendas), il faudrait peut-être envisager d'ajouter un peu de cache ?
lguerin force-pushed wip/72774-toulouse-maelis-bookings from 1b8c92a35f to dd132ffa9d 2023-01-05 15:31:37 +01:00 Compare
lguerin reviewed 2023-01-05 15:35:16 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -1616,0 +1770,4 @@
and day_id in requested_bookings
and action in ['ADD_PRES_PREVI', 'ADD_PRES_REAL', 'DEL_ABSENCE']
):
booked = action
lguerin commented 2023-01-05 15:35:16 +01:00
Author
Owner
Copy Link

J'applique l'action renvoyée par le WS getPersonScheduleList, en vérifiant quand meme qu'elle est pertinente

J'applique l'action renvoyée par le WS getPersonScheduleList, en vérifiant quand meme qu'elle est pertinente
fpeters commented 2023-01-10 14:38:27 +01:00
Owner
Copy Link

test la phrase pas terminée elle s'arrête.

test la phrase pas terminée elle s'arrête.
lguerin reviewed 2023-01-05 15:35:23 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -1616,0 +1776,4 @@
and day_id not in requested_bookings
and action in ['DEL_PRES_PREVI', 'DEL_PRES_REAL', 'ADD_ABSENCE']
):
booked = action
lguerin commented 2023-01-05 15:35:22 +01:00
Author
Owner
Copy Link

idem

idem
lguerin reviewed 2023-01-05 15:36:37 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -1616,0 +1675,4 @@
}
color = 'white'
if booking['prefill']:
color = 'green'
lguerin commented 2023-01-05 15:36:37 +01:00
Author
Owner
Copy Link

Avec ce qu'on a comme info (scheduledPresence et/ou realPresence), seulement 2 couleurs possibles (blanc=non réservé, vert=réservé)
S'il en faut plus, il me faut le mode d'emploi :)

Avec ce qu'on a comme info (scheduledPresence et/ou realPresence), seulement 2 couleurs possibles (blanc=non réservé, vert=réservé) S'il en faut plus, il me faut le mode d'emploi :)
nroche reviewed 2023-01-05 16:08:06 +01:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -1616,0 +1654,4 @@
}
response = self.call('Activity', 'getPersonScheduleList', **payload)
result = serialize_object(response)
for result_data in result:
nroche commented 2023-01-05 16:08:06 +01:00
Owner
Copy Link

Visiblement Sigec renvoie une réponse vide plutôt qu'une liste vide quand un enfant n'a pas d'inscription (ou de calendrier).
Je mettrais :
for result_data in result or []:

Visiblement Sigec renvoie une réponse vide plutôt qu'une liste vide quand un enfant n'a pas d'inscription (ou de calendrier). Je mettrais : `for result_data in result or []:`
lguerin marked this conversation as resolved
nroche approved these changes 2023-01-05 16:11:01 +01:00
nroche left a comment
Owner
Copy Link

Ack de complaisance parce que ça presse.
(J'ai relu/testé la partie affichage du calendriers livrer en WIP hier, mais j'ai juste parcouru la partie réservation qui a suivi)

Ack de complaisance parce que ça presse. (J'ai relu/testé la partie affichage du calendriers livrer en WIP hier, mais j'ai juste parcouru la partie réservation qui a suivi)
lguerin force-pushed wip/72774-toulouse-maelis-bookings from dd132ffa9d to 5c40ae304d 2023-01-05 16:17:13 +01:00 Compare
lguerin merged commit 5c40ae304d into main 2023-01-05 16:37:14 +01:00
lguerin deleted branch wip/72774-toulouse-maelis-bookings 2023-01-05 16:37:14 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
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#19
No description provided.
Delete Branch "wip/72774-toulouse-maelis-bookings"

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?