Skip to content

WIP refactor to use admin namespace for agents dashboard

Vincent Agnano requested to merge refactor/admin-namespace into master

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.

⚠️ j'aimerais bien merger cette PR rapidement parce que c'est vite le chaos à rebase si ça diverge de master

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.

Merge request reports