entrouvert
/
chrono
15
1
Fork 0
Code Issues Pull Requests 10 Releases Wiki Activity

Agenda virtuel : ajouter des infos sur les agendas concernés dans les appels à datetime (#78064) #92

Merged
ecazenave merged 2 commits from wip/78064-virt-infos into main 2023-06-14 12:11:17 +02:00
Conversation 1 Commits 2 Files Changed 3
ecazenave commented 2023-06-01 15:40:15 +02:00
Owner
There is no content yet.
ecazenave added 2 commits 2023-06-01 15:40:15 +02:00
ecazenave force-pushed wip/78064-virt-infos from 01774eb6e7 to e439bb2131 2023-06-01 15:41:51 +02:00 Compare
ecazenave force-pushed wip/78064-virt-infos from e439bb2131 to 38455e39c3 2023-06-01 16:27:11 +02:00 Compare
ecazenave force-pushed wip/78064-virt-infos from 38455e39c3 to c7b78acae9 2023-06-01 16:46:03 +02:00 Compare
ecazenave force-pushed wip/78064-virt-infos from c7b78acae9 to c2275376f8 2023-06-01 17:02:04 +02:00 Compare
ecazenave commented 2023-06-01 17:30:12 +02:00
Poster
Owner

Le truc est la redéfinition de unique_slots dans le cas des agendas virtuel : dans ce cas on ne veut pas laisser à get_all_slots le soin de dédupliquer les créneaux (agenda.get_all_slots(unique=False) dans le cas virtuel). On les veut tous, on retient pour chaque créneau les agenda réels qui le propose et on déduplique en même temps.

(le premier qui me dit ça aurait pu se gérer directement dans get_all_slots aura droit à mon mépris éternel, méthode déjà beaucoup trop compliquée)

Je pensais activer ce comportement via une variable de query string mais finalement ça me semble bon à avoir par défaut sur les agendas virtuels pour en faciliter l'usage. Ça permet aussi d'avoir d'un coup d'un seul moults tests qui font tourner ce nouveau code.

Le truc est la redéfinition de unique_slots dans le cas des agendas virtuel : dans ce cas on ne veut pas laisser à get_all_slots le soin de dédupliquer les créneaux (agenda.get_all_slots(unique=False) dans le cas virtuel). On les veut tous, on retient pour chaque créneau les agenda réels qui le propose et on déduplique en même temps. (le premier qui me dit ça aurait pu se gérer directement dans get_all_slots aura droit à mon mépris éternel, méthode déjà beaucoup trop compliquée) Je pensais activer ce comportement via une variable de query string mais finalement ça me semble bon à avoir par défaut sur les agendas virtuels pour en faciliter l'usage. Ça permet aussi d'avoir d'un coup d'un seul moults tests qui font tourner ce nouveau code.
ecazenave changed title from WIP: wip/78064-virt-infos to wip/78064-virt-infos 2023-06-01 17:30:18 +02:00
ecazenave changed title from wip/78064-virt-infos to Agenda virtuel : ajouter des infos sur les agendas concernés dans les appels à datetime (#78064) 2023-06-01 17:30:56 +02:00
vdeniaud approved these changes 2023-06-14 11:02:33 +02:00
ecazenave force-pushed wip/78064-virt-infos from c2275376f8 to b10501d6e2 2023-06-14 11:50:51 +02:00 Compare
ecazenave force-pushed wip/78064-virt-infos from b10501d6e2 to a303d359d8 2023-06-14 11:52:10 +02:00 Compare
ecazenave merged commit a303d359d8 into main 2023-06-14 12:11:17 +02:00
ecazenave deleted branch wip/78064-virt-infos 2023-06-14 12:11:18 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
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#92
There is no content yet.
Delete Branch "wip/78064-virt-infos"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?