Séparer la homepage de la première page de recherche de rdv #4814
Conversation
There was a problem hiding this comment.
🙇 merci ! c’est un bout du code vraiment pas agréable.
content de voir un niveau d’indirection des vues disparaître 👍
j’ai testé en local avec succès.
On pourra éventuellement aller plus loin en mettant des contraintes domaines dans les routes directement cf https://stackoverflow.com/questions/4207657/rails-routing-to-handle-multiple-domains-on-single-application
on pourra éventuellement faire
root "dsfr/rdv_mairie/homepage", constraints: DomainConstraint.new('RDV_MAIRIE')
root "search#search_rdv" # pour les autres domainesou qqch comme ça
There was a problem hiding this comment.
C'est un bon début de clarification, je vais essayer d'aller plus loin en déplaçant l'action home dans StaticPagesController, et cessant de considérer que address_selection est une step de la prise de RDV, parce que sur 2 instances sur 3, on a pas de sélection d'adresse.
🚀
| @@ -332,9 +331,9 @@ def format_redirect_params(params) | |||
| get "presentation_agent" => "static_pages#presentation_for_agents" | |||
|
|
|||
| resources :lieux, only: %i[index show] | |||
There was a problem hiding this comment.
On peut supprimer cette route qui n'avait pas été supprimée dans #2617, où ::LieuxController a été supprimé. 😉
Contexte
Cette PR sépare en deux actions de controllers les routes de la homepage et du parcours de rdv via le path
/prendre_rdv.Elle me sera utile pour la PR sur l'oauth pour pouvoir appliquer de la logique spécifique à la homepage sans la mélanger à celle de la recherche de rdv.
Mais je pense qu'elle pourra aussi être utile pour clarifier d'autres cas où on distingue la page d'accueil et la page de prise de rdv, notamment #4808.
Solution
Au passage on supprime un niveau d'indirection créé par la vue
app/views/search/_address_selection.html.slim