plages libres, permettre la mise à jour des plages d'une semaine type #78084 #101
lguerin
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/78084-plages-libres-permettre-la-mise-"
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?
@ -107,3 +107,3 @@with CaptureQueriesContext(connection) as ctx:resp = app.post_json(fillslots_url, params=params)assert len(ctx.captured_queries) in [12, 13]assert len(ctx.captured_queries) in [14, 15]Comportement un peu étrange, le code n'ajoute pas de requête mais il y a un effet de bord qui déclenche les deux UPDATE lignes 1638 et 1650 de api/views.py. Je ne sais pas pourquoi elles n'étaient pas déclenchées.
@ -1740,0 +1795,4 @@params['end_time'] = '14:00'resp = app.post_json(fillslots_url, params=params)assert Booking.objects.count() == 15On suit le comportement ajouté par #61066, on se retrouve donc avec un historique des réservations annulées en base. Il faudrait remonter au besoin fonctionnel qui a conduit à #61066 pour statuer ici ?
On garde les annulations, parce qu'on a besoin de remonter cette info dans le calendrier (la superbe couleur jaune)
Mais il faudrait demander à stef et/ou cédric, dans quel cas, pour des rdv crèche, on veut voir remonter des annulations dans le calendrier (les points suivants sont exclusifs):
Je n'ai pas eu de besoin jusqu'à présent de savoir quelle été la réservation avant la modification de celle-ci.
Cela voudrait dire que l'on aurait une barre jaune (ancien créneau réservé) et la verte (nouveau créneau, actuel).
Dans le cas où je modifie 4 fois mon créneau sur la journée, si on décide d'afficher la barre jaune, ne prendre que le créneau modifié, ceux antérieurs on s'en moque.
Par contre, lorsque l'on avait une réservation qui a été annulée (et non remplacée, c'est à dire pas modifiée, mais bel et bien supprimée) il est important de pouvoir l'afficher (barre jaune) et par la suite les quantifier.
==> ceci sert à pouvoir valoriser le travail effectué (poser une réservation à un moment est du travail, ne plus voir cette résa annulée aurait pour conséquence d'oublier le travaille effectué).
En conclusion, je pense qu'il serait bien d'avoir cette visue en jaune pour les 2 cas, résa modifiée et résa annulée.
En terme de gestion cela peut être intéressant :
"Pourquoi j'ai été facturé 1 heure de plus ?
_ et bien vous étiez réservé de 9h à 18h et non de 8h à 18h..
_ ah non
_ ah si si... vous aviez réservé de 8h à 18h et vous l'avez modifiée de 9h à 18h".
Fin du débat.
Sur jabber j'ai objecté à Cédric que si on ne loggait que la dernière annulation on avait pas de manière fiable de dire « vous aviez réservé de telle heure à telle heure » (il peut y avoir un cycle résa -> modif -> modif avec retour à l'horaire initial qui crée un affichage trompeur).
Le truc qu'on pourrait imaginer c'est une page dédiée à l'historique d'une réservation (petit affichage pour dire « cette résa a été modifiée » et clic dessus et on a toutes les infos).
Mais le plus important c'est que Cédric est d'accord pour avoir ça dans une seconde phase :)
Pas d'affichage des annulations pour le moment, donc.
Je fais un patch qui supprime au lieu d'annuler, ou on le garde comme ça ?
ok pour supprimer au lieu d'annuler
Et j'ai à nouveau embêté Cédric, la conclusion c'est qu'on n'enregistre jamais les réservations annulées lors de la réservation d'une semaine type (même dans le futur, et même en cas de changement de jour).
seulement dans le cas plage libre, ou dans tous les cas ?
vu de vive voix, a priori ça ne concerne que le cas plage libre, à confirmer pour le cas usuel avec stef, cédric et mik (qui utilise cette api pour les seniors (?))
a79800f880to4c1f31e8d4WIP: plages libres, permettre la mise à jour des plages d'une semaine type #78084to plages libres, permettre la mise à jour des plages d'une semaine type #78084plages libres, permettre la mise à jour des plages d'une semaine type #78084to WIP: plages libres, permettre la mise à jour des plages d'une semaine type #78084(pour dans le futur avoir la vue des résas annulées, j'archive le patch à utiliser)(vu avec Cédric, normalement on en aura jamais besoin)4c1f31e8d4to3a16e43faf3a16e43fafto31a036f02331a036f023to81906bd124WIP: plages libres, permettre la mise à jour des plages d'une semaine type #78084to plages libres, permettre la mise à jour des plages d'une semaine type #7808481906bd124to6740b56732