entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 4 Releases Wiki Activity

Revoir la gestion des temps ennemis pour les évènements récurrents (#73904) #32

Merged
vdeniaud merged 4 commits from wip/73904-Revoir-la-gestion-des-temps-enne into main 2023-02-13 10:24:08 +01:00
Conversation 1 Commits 4 Files Changed 5 +340 -28
vdeniaud commented 2023-02-01 13:56:30 +01:00
Owner
Copy Link

Un peu de temps perdu en essayant d'adapter le comportement actuel mais en fait non, ce mécanisme de réservation partielle vient se rajouter sur l'existant.

C'est à dire qu'on garde d'un côté la partie « les évènements d'un même appel ne doivent pas se chevaucher » et qu'ici on ajoute « faire attention aux réservations déjà existantes ».

Le côté « avertir de cette situation avant la réservation » c'est un booléen reçu pour chaque évènement dans la réponse qui liste les évènements, charge au widget d'afficher « attention, cet évènement ne pourra pas être réservé en totalité ».

Un peu de temps perdu en essayant d'adapter le comportement actuel mais en fait non, ce mécanisme de réservation partielle vient se rajouter sur l'existant. C'est à dire qu'on garde d'un côté la partie « les évènements d'un même appel ne doivent pas se chevaucher » et qu'ici on ajoute « faire attention aux réservations déjà existantes ». Le côté « avertir de cette situation avant la réservation » c'est un booléen reçu pour chaque évènement dans la réponse qui liste les évènements, charge au widget d'afficher « attention, cet évènement ne pourra pas être réservé en totalité ».
vdeniaud commented 2023-02-01 13:57:45 +01:00
Author
Owner
Copy Link

0001 et 0002 n'ont rien à voir avec la choucroute mais corrigent des manquements sur les API concernées.

0001 et 0002 n'ont rien à voir avec la choucroute mais corrigent des manquements sur les API concernées.
lguerin reviewed 2023-02-06 09:33:28 +01:00
tests/api/datetimes/test_recurring_events.py Outdated
@ -568,0 +679,4 @@
assert resp.json['data'][2]['has_booking_overlaps'] is False
# create one booking on second day
event = Event.objects.get(start_datetime__date='2021-09-22', primary_event=event_13_15)
lguerin commented 2023-02-06 09:29:33 +01:00
Owner
Copy Link

Tu pourrais ajouter un test avec une réservation posée sur une occurence de agenda 1 ? Pour qu'on sache si ça remonte bien en has_booking_overlaps ?

Tu pourrais ajouter un test avec une réservation posée sur une occurence de agenda 1 ? Pour qu'on sache si ça remonte bien en has_booking_overlaps ?
vdeniaud commented 2023-02-06 11:45:25 +01:00
Author
Owner
Copy Link

OK mais ça ne fonctionne pas comme ça, le test montre que ça ne remonte pas en has_booking_overlap (ce qui me paraît logique, si on permet de réserver les occurrences de l'évènement A et qu'il se trouve qu'une est déjà réservée, on n'a pas envie d'afficher de message « attention tout ne pourra pas etre réservé »).

OK mais ça ne fonctionne pas comme ça, le test montre que ça ne remonte pas en has_booking_overlap (ce qui me paraît logique, si on permet de réserver les occurrences de l'évènement A et qu'il se trouve qu'une est déjà réservée, on n'a pas envie d'afficher de message « attention tout ne pourra pas etre réservé »).
lguerin commented 2023-02-06 11:50:27 +01:00
Owner
Copy Link

oui ok, logique

oui ok, logique
lguerin marked this conversation as resolved
vdeniaud force-pushed wip/73904-Revoir-la-gestion-des-temps-enne from 1fcfb7e110 to 7872422748 2023-02-06 11:43:43 +01:00 Compare
lguerin approved these changes 2023-02-06 11:50:50 +01:00
vdeniaud force-pushed wip/73904-Revoir-la-gestion-des-temps-enne from 7872422748 to df9e60149f 2023-02-06 11:57:18 +01:00 Compare
vdeniaud merged commit df9e60149f into main 2023-02-13 10:24:08 +01:00
vdeniaud deleted branch wip/73904-Revoir-la-gestion-des-temps-enne 2023-02-13 10:24:08 +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/chrono#32
No description provided.
Delete Branch "wip/73904-Revoir-la-gestion-des-temps-enne"

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?