Skip to content

extract Sectors from Zones: migration, admin CRUD, user search

Vincent Agnano requested to merge feature/sectorisation-1-extract-sectors into master

Created by: adipasquale

https://trello.com/c/GhKkwtPv/1107-secto-extraction-secteurs

migration de la DB :

stitched_2020_10_07-10_51_11

Cette PR est principalement une préparation pour l'ajout du support d'attributions Agents et de Zones rues à venir. La principale modif fonctionnelle de cette PR est qu'elle permet des secteurs se chevauchant. Il peut y avoir deux secteurs différents qui ont chacun une zone pour la même commune. Une commune peut donc maintenant être rattachée à plusieurs organisations.

nouveaux écrans :

sectors#index

Screenshot_2020-10-07_at_10 56 50

sectors#show

Screenshot_2020-10-07_at_10 57 09

sectors/index_map

Screenshot_2020-10-07_at_10 57 22

détails

  • la PR est longue mais c'est beaucoup de CRUD dans l'admin, rien de très intéréssant. je pense que c'est plus utile de tester la review app que de parcourir en detail le code.
  • j'ai extrait un service SectoriseAddressService qui prend en entrée une adresse (pour l'instant juste un departement et un city_code) et en sortie donne la liste des zones, des secteurs et des orgas associées.
  • supprime la page admin/departement/organisations#show qui permettait de définir les human_id des organisations, plus utilisé pour l'instant.
  • imbrique les routes et vues des Zones sous les secteurs. On ne peut plus voir la liste de toutes les Zones d'un département, il faut d'abord sélectionner un Secteur.
  • le module d'import de fichier ne demande plus des organisation_id mais des sector_id. On ne peut donc plus définir toute la secto au travers d'un seul fichier. Il faut d'abord créer des Secteurs manuellement dans l'interface, puis éventuellement importer un fichier de zones qui contient les communes définissant chaque secteur.
  • la migration et le rollback ont été testés sur un dump de la prod le 19/10

Test review app

Merge request reports