Skip to content

Ajout d’index d’unicité en base (encore)

Vincent Agnano requested to merge more-unique-indices into master

Created by: n-b

idem #1426, annulé dans #1434, parce que ça échouait à la mep. On n’a pas de baddata, mais l’index pour les human_id des organisation se basait sur les null alors qu’il y a en base des chaines vides. J’ai fait comme pour #1425: on commence par remplacer les null par des "", et on fait l’index uniquement sur les non-vides.

Note au passage: j’avais fait tourner scripts/scalingo_dump.sh pour tester en local, mais j’avais omis de faire rails db:migrate sur ces données 🤦 .

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