Project

General

Profile

Development #42629

Sur un guichet permettre d'ajouter une exception avant une période horaire

Added by Emmanuel Cazenave about 2 months ago. Updated 17 days ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
07 May 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Sur un agenda très sollicité auquel on ajouterait un guichet, on ne veux pas qu'une réservation puisse être faite sur une plage horaire que l'on fermera quelques secondes plus tard au moyen d'une exception.

0001-agendas-manager-can-add-exception-even-if-not-timepe.patch View (13 KB) Lauréline Guerin, 01 Jun 2020 10:12 AM

Associated revisions

Revision a65930a1 (diff)
Added by Lauréline Guérin 20 days ago

agendas: manager can add exception even if not timeperiod defined (#42629)

History

#2 Updated by Emmanuel Cazenave about 2 months ago

Pour modérer l'intérêt de ce ticket, il y a un contournement assez simple.

Renseigner une période horaire fictive très courte par exemple le lundi de 10h00 à 10h01, comme comme c'est plus court que la durée de n'importe quel type de rendez-vous (dans la vie la vraie), aucune réservation ne peut-être faite. Puis renseigner les exception, et revenir sur les périodes horaires pour y mettre les vraies.

#3 Updated by Benjamin Dauvergne about 2 months ago

Ne serait-ce pas plus clair d'avoir un champ "enabled" sur un guichet, qu'il soit désactivé au départ le temps de le configurer ?

#4 Updated by Emmanuel Cazenave about 2 months ago

Oui et ça pourrait avoir des cas d'usages plus large du coup, genre préparer un tas de guichets en plus et les 'mettre en service' ultérieurement.

Mais méfiance sur les cas d'usage théoriques.

#5 Updated by Frédéric Péters about 2 months ago

  • Related to Bug #42783: ne plus empêcher de poser des exceptions quand des rendez-vous existent déjà added

#6 Updated by Frédéric Péters about 2 months ago

  • Related to deleted (Bug #42783: ne plus empêcher de poser des exceptions quand des rendez-vous existent déjà)

#7 Updated by Lauréline Guerin about 1 month ago

La duplication des guichets ajouté récemment répond déjà partiellement au cas d'usage décrit.

Du coup je propose de juste permettre l'ajout d'exceptions même s'il n'y a pas de plages horaires, sans mécanisme d'activation d'un guichet.

#8 Updated by Lauréline Guerin about 1 month ago

  • Assignee set to Lauréline Guerin

#9 Updated by Lauréline Guerin about 1 month ago

#10 Updated by Emmanuel Cazenave 21 days ago

Je comprends pas pourquoi il y a autant de code de tests supprimé dans ton patch.

#11 Updated by Lauréline Guerin 21 days ago

Comme il n'y a plus besoin de créer une TimePeriod pour poser une TimePeriodException, j'ai enlevé les clicks pour créer une TimePeriod dans tous les tests qui ne font que créer des Exceptions.

Dans test_meetings_agenda_add_time_period_exception j'ai remplacé la partie "création de data via l'interface" par des créations d'objet. Le test est plus rapide à tourner.
Sachant que toutes ces étapes sont répétées quand quasiment tous les tests, donc elles sont déjà couvertes par le reste.

#12 Updated by Emmanuel Cazenave 21 days ago

  • Status changed from Solution proposée to Solution validée

Lauréline Guerin a écrit :

Comme il n'y a plus besoin de créer une TimePeriod pour poser une TimePeriodException, j'ai enlevé les clicks pour créer une TimePeriod dans tous les tests qui ne font que créer des Exceptions.

Cool, j'aime quand on fait le ménage.

#13 Updated by Lauréline Guerin 20 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit a65930a150ee23c8992f87dbe66e5fd26fe1c6a5
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon Jun 1 10:11:42 2020 +0200

    agendas: manager can add exception even if not timeperiod defined (#42629)

#14 Updated by Frédéric Péters 17 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF