Skip to content

Permettre de marquer un lieu comme fermé

Vincent Agnano requested to merge disable-lieux into master

Created by: n-b

close #1341 (closed)

  • J’ajoute un simple booleen en base. J’ai hésité à mettre disabled_at, mais ça me semblait relativement peu utile, avec PaperTrail, on est couverts au cas où.
  • Le principal, c’est que les lieux fermés n’apparaissent plus
    • à la recherche d’un créneau
    • dans le menu de nouveau Rdv par un agent
    • dans le menu de nouvelle Plage
  • J’ajoute un tag “Fermé” dans les listes et les détails des Rdv/PlageOuverture existants associés à ce lieu.
  • J’avais envie de refaire la liste des lieux (avec un champ de recherche et des onglets ouverts/fermés) et de faire une liste globale, cherchable, des plages d'ouverture de toutes l'organisation. C’est un peu de boulot pour ce deuxième point
  • il manque des tests, au moins d’intégration, mais c’est déjà relisible

Checklist avant review:

  • reparcourir le code rapidement pour voir les problèmes évidents (fichiers touchés inutilement, debug logs qui trainent…).
  • Tester la fonctionnalité sur la review app

Merge request reports