Skip to content

Display papertrail versions of all the things

Vincent Agnano requested to merge versions-versions-versions into recette

Created by: n-b

fixes #1675 (closed)

  • Plutôt que de ne le faire que pour les usagers, je l’ai fait pour tous les objets historisés: Rdv, User, Agents, Organisations, Motifs, et WebhookEndpoints.
  • J’ai supprimé le mécanisme d’affichage via une requête ajax. Moins de JS, moins de bug.
  • Par contre, j’ai mis en place un peu de fragment caching. Avec 5 frontaux web, on aura pas forcément beaucoup de hits, mais c’est déjà ça; je crois qu’on aurait à gagner à en mettre un peu partout, peut-être qu’il faut réfléchir à installer un redis.

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