Isolate agent dashboard to use admin routes prefix
Created by: adipasquale
https://trello.com/c/vvPZpGHL/1058-isoler-les-urls-de-lespace-agent
Beaucoup de code déplacé, très peu modifié.
Je ne pense pas que ca vaille le coup (ni que ce soit humainement possible) de regarder tout le code de cette PR attentivement, mais ça serait cool de faire un bon tour sur la review app.
Les points où il faut faire attention
- tous les path helpers de l'espace agent sont maintenant préfixés
admin_
. c'est verbeux mais ca permettra d'etre explicite et d'eviter les conflits - il faut préfixer les resources lorsqu'on appelle simple_form_for pour qu'il arrive à trouver le bon path helper
simple_form_for([:admin, @organisation])
- j'ai du relancer
brakeman
, un outil d'analyse statique du code que je ne savais meme pas qu'on avait.
Release process
J'ai tout mis dans une seule PR pour aller au bout de ce boulot fatigant et repetitif, mais je pense deployer petit bout par petit bout. Commit par commit ca ferait beaucoup, je pensais faire des sous groupes en cherry-pickant par ex 5 commits par 5, release en journée, attendre une heure puis release l'autre bloc.
qu'en penses-tu @yaf ?
Notes
En passant sur tout ce code j'ai pu remarquer qu'il y a des controlleurs qui me paraissent un peu faire doublon notamment ceux scopés dans le module admin/organisations
. ca complexifie un peu inutilement, on pourrait probablement s'en passer et utiliser le controlleur correspondant scopé dans admin tout court.