Skip to content

Sectorisation par Site à l'échelle de la commune

Vincent Agnano requested to merge feature/sectorisation-site into master

Created by: adipasquale

https://trello.com/c/3poo0MxG/859-sectorisation-par-site

⚠️ Grosse PR ~1500 lignes + 1000l. de seeds et CSV.

ℹ️ À lire absolument commit par commit

  • Les deux premiers commits sont des refactos qui pourraient être faits dans des PRs indépendantes.
  • Puis il y a deux migrations : la création de Zones et l'ajout de organisations#human_id
  • J'ajoute ensuite plein de seeds (900 zones depuis un CSV).
  • Ensuite j'ajoute des interfaces agents admin departement pour modifier les Zones en CRUD classique
  • Puis une interface qui permet de modifier tous les organisations.human_id du departement d'un coup
  • Puis un service pour importer des fichiers de zones aux formats CSV ou XLS + l'interface qui permet de l'utiliser
  • Puis une vue carte un peu hacky avec mapboxgl

Zones ameliorables :

  • contexte pundit pourrait etre changé pour inclure le departement + ameliorer la policy des zones
  • AgentDepartementAuthController pourrait ne pas heriter de AgentAuthController (il n'y a pas de current_organisation dans le departement)

choix techniques:

  • dans la table zones j'ai préféré utiliser une string plutot qu'un entier énuméré. je trouve ca extremement utile de pouvoir lire les valeurs dans la table SQL sans avoir a avoir la reference du code sous les yeux. ca va un peu a l'encontre du reste de la codebase.

Merge request reports