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

manager: forbid multiple events on same day in partial bookings agenda (#79112) #108

Merged
vdeniaud merged 1 commits from wip/79112-plages-libres-empecher-de-config into main 2023-07-03 16:29:53 +02:00
Conversation 2 Commits 1 Files Changed 5 +189 -7
vdeniaud commented 2023-06-27 17:15:39 +02:00
Owner
Copy Link

On pourrait aussi commencer à jouer avec les ExclusionConstraint introduites en Django 3 pour vraiment garantir tout ça (https://docs.djangoproject.com/en/3.2/ref/contrib/postgres/constraints/), mais ça ne remplacerait pas les vérifs en amont ici je pense.

On pourrait aussi commencer à jouer avec les ExclusionConstraint introduites en Django 3 pour vraiment garantir tout ça (https://docs.djangoproject.com/en/3.2/ref/contrib/postgres/constraints/), mais ça ne remplacerait pas les vérifs en amont ici je pense.
vdeniaud force-pushed wip/79112-plages-libres-empecher-de-config from 4c337c22e3 to 42b81dbb00 2023-06-27 17:28:05 +02:00 Compare
vdeniaud changed title from WIP: manager: forbid multiple events on same day in partial bookings agenda (#79112) to manager: forbid multiple events on same day in partial bookings agenda (#79112) 2023-06-27 17:41:37 +02:00
lguerin commented 2023-06-30 09:02:26 +02:00
Owner
Copy Link

Donc on ne peut pas avoir des plages d'ouverture avec une pause à midi ? Genre 8H-12H et 14H-18H ?

Donc on ne peut pas avoir des plages d'ouverture avec une pause à midi ? Genre 8H-12H et 14H-18H ?
vdeniaud commented 2023-07-03 15:03:26 +02:00
Author
Owner
Copy Link

Donc on ne peut pas avoir des plages d'ouverture avec une pause à midi ? Genre 8H-12H et 14H-18H ?

Vu avec Cédric, c'est un cas à la marge dont on ne doit pas s'encombrer pour la V1.

Quand bien même, si on se projette dans ce futur, le code évoluera vers « interdire les évènements superposés », je pense que ça donnera une surcouche au patch actuel (et pas un revert).

En attendant c'est plus simple de coder le pointage, le CSS d'affichage...

Donc pour moi ce patch un peu compliqué vaut le coup.

> Donc on ne peut pas avoir des plages d'ouverture avec une pause à midi ? Genre 8H-12H et 14H-18H ? Vu avec Cédric, c'est un cas à la marge dont on ne doit pas s'encombrer pour la V1. Quand bien même, si on se projette dans ce futur, le code évoluera vers « interdire les évènements superposés », je pense que ça donnera une surcouche au patch actuel (et pas un revert). En attendant c'est plus simple de coder le pointage, le CSS d'affichage... Donc pour moi ce patch un peu compliqué vaut le coup.
vdeniaud force-pushed wip/79112-plages-libres-empecher-de-config from 42b81dbb00 to 4834743c6d 2023-07-03 15:42:21 +02:00 Compare
lguerin approved these changes 2023-07-03 16:28:39 +02:00
vdeniaud merged commit 4834743c6d into main 2023-07-03 16:29:53 +02:00
vdeniaud deleted branch wip/79112-plages-libres-empecher-de-config 2023-07-03 16:29:53 +02: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#108
No description provided.
Delete Branch "wip/79112-plages-libres-empecher-de-config"

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?