betagouv / collectif-objets

Recenser notre patrimoine pour le protéger
https://collectif-objets.beta.gouv.fr
8 stars 6 forks source link

Correctif du bug d'import constaté sur la commune de La Versanne #1350

Closed bricedurand closed 1 month ago

bricedurand commented 1 month ago

Constat : Les objets de La Versanne n'étaient pas importés dans Collectif Objets, alors qu'à priori il rentrent dans le périmètre.

En lançant l'import en local, je me suis rendu compte qu'il y avait un souci avec la suppression (en l'occurence c'est un soft-delete) de recensements lorsqu'un objet est déplacé dans une autre commune.

La contrainte d'unicité validates :objet_id, uniqueness: { scope: :dossier_id } que l'on avait ajoutée suite au re-recensement bloquait la suppression car il pouvait y avoir d'autres recensements supprimés du même dossier avec un objet_id à nil. J'ai donc retiré la contrainte pour les recensements supprimés.

Pour info, ces recensements supprimés sont affiché sur la page de la commune côté conservateur par cet encart : Screenshot 2024-09-09 at 12 20 06

bricedurand commented 1 month ago

Bonus : mise à jour de certaines libs et ajout du fichier .tool-versions pour spécifier les versions de Ruby et Node à asdf