Skip to content

Add Rdv revoked status

Vincent Agnano requested to merge rdv-status-revoked into master

Created by: n-b

Refs #1551 (closed)

C’est fonctionnel (je crois): Il y a une nouvelle valeur dans le code, qui est gérée dans les menus, les stats et utilisée pour les contenus des notifications. J’ai quelques améliorations à faire au passage, dans une PR suivante peut-être:

  • Utiliser un véritable enum en PG
    • et ajouter un index en base
  • Corriger les quelques bugs visuels du menu déroulant
  • Corriger les bugs visuels du titre de la carte du Rdv
  • Utiliser un seul élément d’interface pour les changements de statut, plutôt que les boutons “annuler” et le menu “Statut”. Ça fait aujourd’hui double emploi, mais il faut sans doute ajuster le texte et le style pour que ça soit clair.

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