Development #62590
Changement d'heure, délai de réservation et agenda virtuel
0%
Description
Dans #62500, un cŕenau qui apparaît libre sur un agenda virtuel alors que non pas libre.
Ça a à voir avec le changement d'heure qui interviendra entre maintenant et la date du faux créneau libre et les délai de réservation posés sur l'agenda réel.
Very fun.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-agendas-wip-62590.patch 0001-agendas-wip-62590.patch ajouté
- Patch proposed changé de Non à Oui
Je pose le patch pour le test unitaire test_virtual_agendas_time_change
qui reproduit bien le problème.
La correction n'est pas bonne, ça fait échouer test_datetimes_api_meetings_agenda_time_change
.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
Le problème est dans get_all_slots qui continue à avoir une notion de start_of_day à un endroit alors que ça n'a pas tellement de sens (le jour du changement d'heure c'est difficile définissable), je ne pense pas que ça serve encore... j'ai un patch mais je regarde encore un peu.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Fichier 0001-api-do-not-move-desk-max_datetime-if-not-on-a-day-bo.patch 0001-api-do-not-move-desk-max_datetime-if-not-on-a-day-bo.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de En cours à Solution proposée
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-api-do-not-move-desk-max_datetime-if-not-on-a-day-bo.patch 0001-api-do-not-move-desk-max_datetime-if-not-on-a-day-bo.patch ajouté
Emmanuel Cazenave a écrit :
Dans #62500, un cŕenau qui apparaît libre sur un agenda virtuel alors que non pas libre.
Description foireuse, le créneau ne devrait pas apparaitre du tout, "masqué" par le maximal_booking_delay de l'agenda réel qui entre en jeu parce que pas de maximal_booking_delay définis sur l'agenda virtuel.
Test mis à jour pour rendre compte de cela, ton fix est toujours ok Benj.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit f60d1498bca9970126bc2ee96708773a46d95dd5 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Tue Mar 8 18:33:57 2022 +0100 api: do not move desk max_datetime if not on a day boundary (#62590)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api: do not move desk max_datetime if not on a day boundary (#62590)