Skip to content

Corrige une 404 à l’affichage de l’historique d’un Rdv si un ancien Lieu a été supprimé

Vincent Agnano requested to merge fix-rdv-versions-404 into master

Created by: n-b

Relevé par Élodie, pas d’Issue associée. Ce n’est pas une iframe, mais ça y ressemble un peu: VersionsController#index retourne du html, qui est inclus en ajax dans le layout de Admin::RdvsController#show. Je n’ai pas rajouté de test, parce que ça demanderait un peu plus de travail : VersionsController n’est actuellement pas testé, et Admin::RdvsController en controller, alors que j’aurais besoin de Capybara.

Après ce commit, le changement dans l’historique affiche l’id de l’objet supprimé, au lieu de planter en essayant de récupérer les détails:

image
  • checklist avant review:
  • reparcourir le code rapidement pour voir les problèmes évidents (fichiers touched inutilement, debug logs qui trainent...).
  • Attendre que les tests soient verts sur la CI
  • Tester la fonctionnalité (si pertinent) sur la review app

Merge request reports