From 551f166873361d9797ae8329b4e9d61d62ef11ef Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Tue, 5 Nov 2024 18:43:50 +0100
Subject: [PATCH 1/7] style(simple routing): update view

---
 .../simple_routing.html.haml                  | 48 +++++++++++--------
 1 file changed, 29 insertions(+), 19 deletions(-)

diff --git a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
index c1a97daa0..057b8cdfe 100644
--- a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
+++ b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
@@ -4,24 +4,34 @@
                     ['Groupes', admin_procedure_groupe_instructeurs_path(@procedure)],
                     ['Routage à partir d’un champ']] }
 
-= render Procedure::InstructeursMenuComponent.new(procedure: @procedure) do
-  - content_for(:title, 'Routage')
-  %h1 Routage à partir d’un champ
-  = form_for :create_simple_routing,
-    method: :post,
-    data: { controller: 'enable-submit-if-checked' },
-    url: create_simple_routing_admin_procedure_groupe_instructeurs_path(@procedure) do |f|
+.container
+  .fr-grid-row
+    .fr-col.fr-col-12.fr-col-md-3
+      .fr-container
+        %ul.fr-btns-group.fr-btns-group--inline-md.fr-ml-0
+          %li
+            = link_to options_admin_procedure_groupe_instructeurs_path, class: 'fr-link fr-icon-arrow-left-line fr-link--icon-left fr-mb-2w fr-mr-2w' do
+              Revenir aux options
 
-    %div{ data: { 'action': "click->enable-submit-if-checked#click" } }
-      .notice
-        Sélectionner le champ à partir duquel créer des groupes d’instructeurs
-      - buttons_content = @procedure.active_revision.simple_routable_types_de_champ.map { |tdc| { label: tdc.libelle, value: tdc.stable_id } }
-      = render Dsfr::RadioButtonListComponent.new(form: f,
-        target: :stable_id,
-        buttons: buttons_content)
+    .fr-col
+      - content_for(:title, 'Routage')
+      %h1 Configuration du routage
+      %h2 Routage à partir d’un champ
+      = form_for :create_simple_routing,
+        method: :post,
+        data: { controller: 'enable-submit-if-checked' },
+        url: create_simple_routing_admin_procedure_groupe_instructeurs_path(@procedure) do |f|
 
-      %ul.fr-btns-group.fr-btns-group--inline-sm
-        %li
-          = link_to 'Retour', options_admin_procedure_groupe_instructeurs_path(@procedure, state: :choix), class: 'fr-btn fr-btn--secondary'
-        %li
-          %button.fr-btn{ disabled: true,  data: { disable_with: 'Création des groupes…', 'enable-submit-if-checked-target': 'submit' } } Créer les groupes
+        .card.fr-pb-0{ data: { 'action': "click->enable-submit-if-checked#click" } }
+          .notice
+            Sélectionner le champ à partir duquel créer des groupes d’instructeurs
+          - buttons_content = @procedure.active_revision.simple_routable_types_de_champ.map { |tdc| { label: tdc.libelle, value: tdc.stable_id } }
+          = render Dsfr::RadioButtonListComponent.new(form: f,
+            target: :stable_id,
+            buttons: buttons_content)
+
+        %ul.fr-btns-group.fr-btns-group--inline-sm
+          %li
+            = link_to 'Annuler', options_admin_procedure_groupe_instructeurs_path(@procedure, state: :choix), class: 'fr-btn fr-btn--secondary'
+          %li
+            %button.fr-btn{ disabled: true,  data: { disable_with: 'Création des groupes…', 'enable-submit-if-checked-target': 'submit' } } Créer les groupes

From b006a87730d4f0913a7518dc03d4cb4d4cded713 Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Thu, 7 Nov 2024 09:08:22 +0100
Subject: [PATCH 2/7] feat(simple routing): add alert info

---
 .../groupe_instructeurs/simple_routing.html.haml       | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
index 057b8cdfe..6ef8dd4ad 100644
--- a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
+++ b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
@@ -17,6 +17,16 @@
       - content_for(:title, 'Routage')
       %h1 Configuration du routage
       %h2 Routage à partir d’un champ
+      .fr-alert.fr-alert--info.fr-mb-3w{ aria: { hidden: true } }
+        %p
+          Vous trouverez ci-dessous une <strong>suggestion</strong> (non exhaustive) de champs issus de votre formulaire. Les groupes d’instructeurs seront créés sur la base du champ que vous aurez sélectionné.
+          %br
+          Vous pourrez ensuite affiner votre routage en ajoutant des groupes sur la base de l’ensemble des <strong>champs « routables »</strong> de votre formulaire, soit des champs de type :
+        %ul
+          - TypeDeChamp.humanized_conditionable_types_by_category.each do |category|
+            %li
+              = category.join(', ')
+
       = form_for :create_simple_routing,
         method: :post,
         data: { controller: 'enable-submit-if-checked' },

From 05b28fb75d93fd711f5fd323936f82410362d0fb Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Fri, 8 Nov 2024 15:08:12 +0100
Subject: [PATCH 3/7] feat(routing): add flash modal after simple routing
 configurated

---
 .../groupes_management_component.html.haml         |  2 ++
 .../groupe_instructeurs_controller.rb              |  3 ++-
 .../_simple_routing_modal.html.haml                | 14 ++++++++++++++
 .../groupe_instructeurs_controller_spec.rb         | 14 +++++++-------
 spec/system/routing/rules_full_scenario_spec.rb    |  2 ++
 5 files changed, 27 insertions(+), 8 deletions(-)
 create mode 100644 app/views/administrateurs/groupe_instructeurs/_simple_routing_modal.html.haml

diff --git a/app/components/procedure/groupes_management_component/groupes_management_component.html.haml b/app/components/procedure/groupes_management_component/groupes_management_component.html.haml
index 7548d6da0..2e6bc12fe 100644
--- a/app/components/procedure/groupes_management_component/groupes_management_component.html.haml
+++ b/app/components/procedure/groupes_management_component/groupes_management_component.html.haml
@@ -53,3 +53,5 @@
 
 - if flash[:routing_mode] == 'custom'
   = render partial: 'custom_routing_modal'
+- elsif flash[:routing_mode] == 'simple'
+  = render partial: 'simple_routing_modal', locals: { procedure: @procedure }
diff --git a/app/controllers/administrateurs/groupe_instructeurs_controller.rb b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
index fa76101ed..43ab472fe 100644
--- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
@@ -88,7 +88,8 @@ module Administrateurs
         defaut.destroy!
       end
 
-      flash.notice = 'Les groupes instructeurs ont été ajoutés'
+      flash[:routing_mode] = 'simple'
+
       redirect_to admin_procedure_groupe_instructeurs_path(@procedure)
     end
 
diff --git a/app/views/administrateurs/groupe_instructeurs/_simple_routing_modal.html.haml b/app/views/administrateurs/groupe_instructeurs/_simple_routing_modal.html.haml
new file mode 100644
index 000000000..d652983a9
--- /dev/null
+++ b/app/views/administrateurs/groupe_instructeurs/_simple_routing_modal.html.haml
@@ -0,0 +1,14 @@
+%dialog{ aria: { labelledby: "fr-modal-title-modal-1" }, role: "dialog", id: "routing-mode-modal", class: "fr-modal fr-modal--opened" }
+  .fr-container.fr-container--fluid.fr-container-md
+    .fr-grid-row.fr-grid-row--center
+      .fr-col-12.fr-col-md-8.fr-col-lg-6
+        .fr-modal__body
+          .fr-modal__header
+            %button.fr-btn.fr-btn--close{ title: "Fermer la fenêtre modale", aria: { controls: "routing-mode-modal" } } Fermer
+          .fr-modal__content
+            %h1#fr-modal-title-modal-1.fr-modal__title
+              %span.fr-icon-arrow-right-line.fr-icon--lg
+              Routage à partir d’un champ
+            .fr-alert.fr-alert--success
+              %h2.fr-alert__title
+                Les groupes instructeurs ont été créés à partir du champ « #{procedure.routing_champs.first} »
diff --git a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
index 7b4e54ba1..37c55a6d5 100644
--- a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
@@ -918,7 +918,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
       it do
         expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure3))
-        expect(flash.notice).to eq 'Les groupes instructeurs ont été ajoutés'
+        expect(flash[:routing_mode]).to eq 'simple'
         expect(procedure3.groupe_instructeurs.pluck(:label)).to match_array(['Paris', 'Lyon', 'Marseille'])
         expect(procedure3.reload.defaut_groupe_instructeur.routing_rule).to eq(ds_eq(champ_value(drop_down_tdc.stable_id), constant('Lyon')))
         expect(procedure3.routing_enabled).to be_truthy
@@ -938,7 +938,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
       it do
         expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure3))
-        expect(flash.notice).to eq 'Les groupes instructeurs ont été ajoutés'
+        expect(flash[:routing_mode]).to eq 'simple'
         expect(procedure3.groupe_instructeurs.pluck(:label)).to include("01 – Ain")
         expect(procedure3.reload.defaut_groupe_instructeur.routing_rule).to eq(ds_eq(champ_value(departements_tdc.stable_id), constant('01')))
         expect(procedure3.routing_enabled).to be_truthy
@@ -958,7 +958,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
       it do
         expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure3))
-        expect(flash.notice).to eq 'Les groupes instructeurs ont été ajoutés'
+        expect(flash[:routing_mode]).to eq 'simple'
         expect(procedure3.groupe_instructeurs.pluck(:label)).to include("Guadeloupe")
         expect(procedure3.reload.defaut_groupe_instructeur.routing_rule).to eq(ds_eq(champ_value(regions_tdc.stable_id), constant('84')))
         expect(procedure3.routing_enabled).to be_truthy
@@ -978,7 +978,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
       it do
         expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure3))
-        expect(flash.notice).to eq 'Les groupes instructeurs ont été ajoutés'
+        expect(flash[:routing_mode]).to eq 'simple'
         expect(procedure3.groupe_instructeurs.pluck(:label)).to include("AD – Andorre")
         expect(procedure3.reload.defaut_groupe_instructeur.routing_rule).to eq(ds_eq(champ_value(pays_tdc.stable_id), constant('AD')))
         expect(procedure3.routing_enabled).to be_truthy
@@ -998,7 +998,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
       it do
         expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure3))
-        expect(flash.notice).to eq 'Les groupes instructeurs ont été ajoutés'
+        expect(flash[:routing_mode]).to eq 'simple'
         expect(procedure3.groupe_instructeurs.pluck(:label)).to include("01 – Ain")
         expect(procedure3.reload.defaut_groupe_instructeur.routing_rule).to eq(ds_in_departement(champ_value(communes_tdc.stable_id), constant('01')))
         expect(procedure3.routing_enabled).to be_truthy
@@ -1018,7 +1018,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
       it do
         expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure3))
-        expect(flash.notice).to eq 'Les groupes instructeurs ont été ajoutés'
+        expect(flash[:routing_mode]).to eq 'simple'
         expect(procedure3.groupe_instructeurs.pluck(:label)).to include("01 – Ain")
         expect(procedure3.reload.defaut_groupe_instructeur.routing_rule).to eq(ds_in_departement(champ_value(epci_tdc.stable_id), constant('01')))
         expect(procedure3.routing_enabled).to be_truthy
@@ -1038,7 +1038,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
       it do
         expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure3))
-        expect(flash.notice).to eq 'Les groupes instructeurs ont été ajoutés'
+        expect(flash[:routing_mode]).to eq 'simple'
         expect(procedure3.groupe_instructeurs.pluck(:label)).to include("01 – Ain")
         expect(procedure3.reload.defaut_groupe_instructeur.routing_rule).to eq(ds_in_departement(champ_value(address_tdc.stable_id), constant('01')))
         expect(procedure3.routing_enabled).to be_truthy
diff --git a/spec/system/routing/rules_full_scenario_spec.rb b/spec/system/routing/rules_full_scenario_spec.rb
index 56e1c7942..094a284a7 100644
--- a/spec/system/routing/rules_full_scenario_spec.rb
+++ b/spec/system/routing/rules_full_scenario_spec.rb
@@ -33,6 +33,8 @@ describe 'The routing with rules', js: true do
     expect(page).to have_text('3 groupes')
     expect(page).not_to have_text('à configurer')
 
+    within("#routing-mode-modal") { click_on "Fermer" }
+
     click_on 'littéraire'
     expect(page).to have_select("groupe_instructeur[condition_form][rows][][targeted_champ]", selected: "Spécialité")
     expect(page).to have_select("groupe_instructeur[condition_form][rows][][value]", selected: "littéraire")

From 4e8200aaf7c5fcab4330592e50f1458702adabe4 Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Fri, 8 Nov 2024 17:08:18 +0100
Subject: [PATCH 4/7] feat(routing): add types of routables champs

---
 .../groupe_instructeurs/simple_routing.html.haml                | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
index 6ef8dd4ad..bdcaa97aa 100644
--- a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
+++ b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
@@ -35,7 +35,7 @@
         .card.fr-pb-0{ data: { 'action': "click->enable-submit-if-checked#click" } }
           .notice
             Sélectionner le champ à partir duquel créer des groupes d’instructeurs
-          - buttons_content = @procedure.active_revision.simple_routable_types_de_champ.map { |tdc| { label: tdc.libelle, value: tdc.stable_id } }
+          - buttons_content = @procedure.active_revision.simple_routable_types_de_champ.map { |tdc| { label: tdc.libelle, value: tdc.stable_id, hint: "[#{I18n.t(tdc.type_champ, scope: 'activerecord.attributes.type_de_champ.type_champs')}]"} }
           = render Dsfr::RadioButtonListComponent.new(form: f,
             target: :stable_id,
             buttons: buttons_content)

From 304fb9ecef0d8b4a6582f66503b5874e49d2236e Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Wed, 13 Nov 2024 09:53:56 +0100
Subject: [PATCH 5/7] feat(simple routing): add options of list tdcs in tooltip

---
 app/components/dsfr/radio_button_list_component.rb   |  2 +-
 .../radio_button_list_component.html.haml            | 12 +++++++++---
 .../groupe_instructeurs/simple_routing.html.haml     |  2 +-
 3 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/app/components/dsfr/radio_button_list_component.rb b/app/components/dsfr/radio_button_list_component.rb
index 566d36653..775b70784 100644
--- a/app/components/dsfr/radio_button_list_component.rb
+++ b/app/components/dsfr/radio_button_list_component.rb
@@ -18,7 +18,7 @@ class Dsfr::RadioButtonListComponent < ApplicationComponent
 
   def each_button
     @buttons.each do |button|
-      yield(*button.values_at(:label, :value, :hint), **button.except(:label, :value, :hint))
+      yield(*button.values_at(:label, :value, :hint, :tooltip), **button.except(:label, :value, :hint, :tooltip))
     end
   end
 end
diff --git a/app/components/dsfr/radio_button_list_component/radio_button_list_component.html.haml b/app/components/dsfr/radio_button_list_component/radio_button_list_component.html.haml
index 8776b11af..f0dd1c403 100644
--- a/app/components/dsfr/radio_button_list_component/radio_button_list_component.html.haml
+++ b/app/components/dsfr/radio_button_list_component/radio_button_list_component.html.haml
@@ -1,8 +1,9 @@
 %fieldset{ class: class_names("fr-fieldset": true, "fr-fieldset--error": error?), 'aria-labelledby': 'radio-hint-element-legend radio-hint-element-messages', role: error? ? :group : nil }
   %legend.fr-fieldset__legend--regular.fr-fieldset__legend
     = content
-
-  - each_button do |label, value, hint, **button_options|
+  - index = 0
+  - each_button do |label, value, hint, tooltip, **button_options|
+    - index += 1
     .fr-fieldset__element
       .fr-radio-group
         = @form.radio_button @target, value, **button_options
@@ -12,7 +13,12 @@
 
             = button_options[:after_label] if button_options[:after_label]
 
-            %span.fr-hint-text= hint if hint
+            - if hint.present?
+              .flex
+                .fr-hint-text= hint
+                - if tooltip.present?
+                  .fr-icon-information-line.fr-icon--sm.ml-1{ 'aria-describedby': "tooltip-#{index}" }
+                  %span.fr-tooltip.fr-placement{ id: "tooltip-#{index}", role: 'tooltip', 'aria-hidden': 'true' }= tooltip
 
   .fr-messages-group{ 'aria-live': 'assertive' }
     - if error?
diff --git a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
index bdcaa97aa..3fb90b18b 100644
--- a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
+++ b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
@@ -35,7 +35,7 @@
         .card.fr-pb-0{ data: { 'action': "click->enable-submit-if-checked#click" } }
           .notice
             Sélectionner le champ à partir duquel créer des groupes d’instructeurs
-          - buttons_content = @procedure.active_revision.simple_routable_types_de_champ.map { |tdc| { label: tdc.libelle, value: tdc.stable_id, hint: "[#{I18n.t(tdc.type_champ, scope: 'activerecord.attributes.type_de_champ.type_champs')}]"} }
+          - buttons_content = @procedure.active_revision.simple_routable_types_de_champ.map { |tdc| { label: tdc.libelle, value: tdc.stable_id, hint: "[#{I18n.t(tdc.type_champ, scope: 'activerecord.attributes.type_de_champ.type_champs')}]", tooltip: tdc.drop_down_options.join(", ")} }
           = render Dsfr::RadioButtonListComponent.new(form: f,
             target: :stable_id,
             buttons: buttons_content)

From 61c8fa4601399f8a49684a9f4409174fd2d1ba68 Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Thu, 14 Nov 2024 12:03:44 +0100
Subject: [PATCH 6/7] refactor(simple routing): move button index in the
 component

---
 app/components/dsfr/radio_button_list_component.rb     |  4 ++--
 .../radio_button_list_component.html.haml              | 10 ++++------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/app/components/dsfr/radio_button_list_component.rb b/app/components/dsfr/radio_button_list_component.rb
index 775b70784..df84138a3 100644
--- a/app/components/dsfr/radio_button_list_component.rb
+++ b/app/components/dsfr/radio_button_list_component.rb
@@ -17,8 +17,8 @@ class Dsfr::RadioButtonListComponent < ApplicationComponent
   end
 
   def each_button
-    @buttons.each do |button|
-      yield(*button.values_at(:label, :value, :hint, :tooltip), **button.except(:label, :value, :hint, :tooltip))
+    @buttons.each.with_index do |button, index|
+      yield(*button.values_at(:label, :value, :hint, :tooltip), **button.merge!(index:).except(:label, :value, :hint, :tooltip))
     end
   end
 end
diff --git a/app/components/dsfr/radio_button_list_component/radio_button_list_component.html.haml b/app/components/dsfr/radio_button_list_component/radio_button_list_component.html.haml
index f0dd1c403..8c58448ad 100644
--- a/app/components/dsfr/radio_button_list_component/radio_button_list_component.html.haml
+++ b/app/components/dsfr/radio_button_list_component/radio_button_list_component.html.haml
@@ -1,12 +1,10 @@
 %fieldset{ class: class_names("fr-fieldset": true, "fr-fieldset--error": error?), 'aria-labelledby': 'radio-hint-element-legend radio-hint-element-messages', role: error? ? :group : nil }
   %legend.fr-fieldset__legend--regular.fr-fieldset__legend
     = content
-  - index = 0
   - each_button do |label, value, hint, tooltip, **button_options|
-    - index += 1
     .fr-fieldset__element
       .fr-radio-group
-        = @form.radio_button @target, value, **button_options
+        = @form.radio_button @target, value, **button_options.except(:index)
         = @form.label @target, value: value, class: 'fr-label' do
           - capture do
             = label
@@ -16,9 +14,9 @@
             - if hint.present?
               .flex
                 .fr-hint-text= hint
-                - if tooltip.present?
-                  .fr-icon-information-line.fr-icon--sm.ml-1{ 'aria-describedby': "tooltip-#{index}" }
-                  %span.fr-tooltip.fr-placement{ id: "tooltip-#{index}", role: 'tooltip', 'aria-hidden': 'true' }= tooltip
+                - if tooltip.present? && button_options[:index]
+                  .fr-icon-information-line.fr-icon--sm.ml-1{ 'aria-describedby': "tooltip-#{button_options[:index]}" }
+                  %span.fr-tooltip.fr-placement{ id: "tooltip-#{button_options[:index]}", role: 'tooltip', 'aria-hidden': 'true' }= tooltip
 
   .fr-messages-group{ 'aria-live': 'assertive' }
     - if error?

From bccef87cbfde76ccb17b0e1e9e0bab58df74bd4c Mon Sep 17 00:00:00 2001
From: Eric Leroy-Terquem <eric.leroyterquem@beta.gouv.fr>
Date: Thu, 14 Nov 2024 17:16:46 +0100
Subject: [PATCH 7/7] wording(routing): updates after UI reviews

---
 .../instructeurs_options_component.html.haml  |  8 ++++----
 app/models/logic/champ_value.rb               |  2 +-
 app/models/type_de_champ.rb                   | 12 ++++++++++++
 .../_custom_routing_modal.html.haml           |  2 +-
 .../_simple_routing_modal.html.haml           |  2 +-
 .../simple_routing.html.haml                  | 19 +++++++++++++------
 .../groupe_instructeurs_controller_spec.rb    |  2 +-
 spec/models/type_de_champ_spec.rb             |  2 +-
 .../routing/rules_full_scenario_spec.rb       | 12 ++++++------
 9 files changed, 40 insertions(+), 21 deletions(-)

diff --git a/app/components/procedure/instructeurs_options_component/instructeurs_options_component.html.haml b/app/components/procedure/instructeurs_options_component/instructeurs_options_component.html.haml
index e4084a801..36f8d1665 100644
--- a/app/components/procedure/instructeurs_options_component/instructeurs_options_component.html.haml
+++ b/app/components/procedure/instructeurs_options_component/instructeurs_options_component.html.haml
@@ -55,14 +55,14 @@
     url: wizard_admin_procedure_groupe_instructeurs_path(@procedure) do |f|
 
     %h1 Configuration du routage
-    %h2 Choix du type de routage
+    %h2 Choix du type de configuration
 
     .card.fr-pb-0{ data: { 'action': "click->enable-submit-if-checked#click" } }
-      %p.fr-mb-0 Routage
+      %p.fr-mb-0 Configuration
       = render Dsfr::RadioButtonListComponent.new(form: f,
         target: :state,
-        buttons: [ { label: 'À partir d’un champ', value: 'routage_simple', hint: 'crée les groupes en fonction d’un champ du formulaire' } ,
-          { label: 'Avancé', value: 'custom_routing', hint: 'libre à vous de créer et de configurer les groupes' }])
+        buttons: [ { label: 'Automatique', value: 'routage_simple', hint: 'crée les groupes automatiquement à partir des valeurs possibles d’un champ du formulaire usager' } ,
+          { label: 'Manuelle', value: 'custom_routing', hint: 'libre à vous de créer et de configurer les groupes en utilisant les champs « routables » du formulaire usager' }])
 
 
     %ul.fr-btns-group.fr-btns-group--inline-sm
diff --git a/app/models/logic/champ_value.rb b/app/models/logic/champ_value.rb
index b63972089..1d9c9858e 100644
--- a/app/models/logic/champ_value.rb
+++ b/app/models/logic/champ_value.rb
@@ -8,11 +8,11 @@ class Logic::ChampValue < Logic::Term
     :decimal_number,
     :drop_down_list,
     :multiple_drop_down_list,
+    :address,
     :communes,
     :epci,
     :departements,
     :regions,
-    :address,
     :pays
   )
 
diff --git a/app/models/type_de_champ.rb b/app/models/type_de_champ.rb
index 0403ac864..b380dc9fe 100644
--- a/app/models/type_de_champ.rb
+++ b/app/models/type_de_champ.rb
@@ -529,6 +529,18 @@ class TypeDeChamp < ApplicationRecord
       .map { |_, v| v.map { "« #{I18n.t(_1, scope: [:activerecord, :attributes, :type_de_champ, :type_champs])} »" } }
   end
 
+  def self.humanized_simple_routable_types_by_category
+    Logic::ChampValue::MANAGED_TYPE_DE_CHAMP_BY_CATEGORY
+      .map { |_, v| v.filter_map { "« #{I18n.t(_1, scope: [:activerecord, :attributes, :type_de_champ, :type_champs])} »" if _1.to_s.in?(SIMPLE_ROUTABLE_TYPES) } }
+      .reject(&:empty?)
+  end
+
+  def self.humanized_custom_routable_types_by_category
+    Logic::ChampValue::MANAGED_TYPE_DE_CHAMP_BY_CATEGORY
+      .map { |_, v| v.filter_map { "« #{I18n.t(_1, scope: [:activerecord, :attributes, :type_de_champ, :type_champs])} »" if !_1.to_s.in?(SIMPLE_ROUTABLE_TYPES) } }
+      .reject(&:empty?)
+  end
+
   def invalid_regexp?
     self.errors.delete(:expression_reguliere)
     self.errors.delete(:expression_reguliere_exemple_text)
diff --git a/app/views/administrateurs/groupe_instructeurs/_custom_routing_modal.html.haml b/app/views/administrateurs/groupe_instructeurs/_custom_routing_modal.html.haml
index cc503f514..4f14875e0 100644
--- a/app/views/administrateurs/groupe_instructeurs/_custom_routing_modal.html.haml
+++ b/app/views/administrateurs/groupe_instructeurs/_custom_routing_modal.html.haml
@@ -8,7 +8,7 @@
           .fr-modal__content
             %h1#fr-modal-title-modal-1.fr-modal__title
               %span.fr-icon-arrow-right-line.fr-icon--lg
-              Routage avancé
+              Configuration manuelle du routage
             .fr-alert.fr-alert--success
               %h2.fr-alert__title
                 Deux groupes par défaut ont été créés
diff --git a/app/views/administrateurs/groupe_instructeurs/_simple_routing_modal.html.haml b/app/views/administrateurs/groupe_instructeurs/_simple_routing_modal.html.haml
index d652983a9..ff817b750 100644
--- a/app/views/administrateurs/groupe_instructeurs/_simple_routing_modal.html.haml
+++ b/app/views/administrateurs/groupe_instructeurs/_simple_routing_modal.html.haml
@@ -8,7 +8,7 @@
           .fr-modal__content
             %h1#fr-modal-title-modal-1.fr-modal__title
               %span.fr-icon-arrow-right-line.fr-icon--lg
-              Routage à partir d’un champ
+              Configuration automatique du routage
             .fr-alert.fr-alert--success
               %h2.fr-alert__title
                 Les groupes instructeurs ont été créés à partir du champ « #{procedure.routing_champs.first} »
diff --git a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
index 3fb90b18b..de4deae6a 100644
--- a/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
+++ b/app/views/administrateurs/groupe_instructeurs/simple_routing.html.haml
@@ -2,7 +2,7 @@
   locals: { steps: [[t('.procedures'), admin_procedures_path],
                     [@procedure.libelle.truncate_words(10), admin_procedure_path(@procedure)],
                     ['Groupes', admin_procedure_groupe_instructeurs_path(@procedure)],
-                    ['Routage à partir d’un champ']] }
+                    ['Configuration automatique du routage']] }
 
 .container
   .fr-grid-row
@@ -16,14 +16,21 @@
     .fr-col
       - content_for(:title, 'Routage')
       %h1 Configuration du routage
-      %h2 Routage à partir d’un champ
+      %h2 Configuration automatique
       .fr-alert.fr-alert--info.fr-mb-3w{ aria: { hidden: true } }
         %p
-          Vous trouverez ci-dessous une <strong>suggestion</strong> (non exhaustive) de champs issus de votre formulaire. Les groupes d’instructeurs seront créés sur la base du champ que vous aurez sélectionné.
-          %br
-          Vous pourrez ensuite affiner votre routage en ajoutant des groupes sur la base de l’ensemble des <strong>champs « routables »</strong> de votre formulaire, soit des champs de type :
+        Vous trouverez ci-dessous une liste de champs de votre formulaire à partir desquels configurer le routage de façon <strong>automatique</strong>. Les groupes d’instructeurs seront créés à partir des valeurs possibles du champ.
+        Seuls les champs suivants sont ouverts à ce mode de configuration :
         %ul
-          - TypeDeChamp.humanized_conditionable_types_by_category.each do |category|
+          - TypeDeChamp.humanized_simple_routable_types_by_category.each do |category|
+            %li
+              = category.join(', ')
+
+        %p
+          Si besoin, vous pourrez ensuite affiner votre configuration de façon <strong>manuelle</strong>, également à partir des champs suivants :
+
+        %ul
+          - TypeDeChamp.humanized_custom_routable_types_by_category.each do |category|
             %li
               = category.join(', ')
 
diff --git a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
index 37c55a6d5..84db0d18f 100644
--- a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
@@ -879,7 +879,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
 
       it do
         expect(response).to have_http_status(:ok)
-        expect(response.body).to include('Choix du type de routage')
+        expect(response.body).to include('Choix du type de configuration')
         expect(procedure.reload.routing_enabled).to be_falsey
       end
     end
diff --git a/spec/models/type_de_champ_spec.rb b/spec/models/type_de_champ_spec.rb
index 3210d5465..b8d4e44f3 100644
--- a/spec/models/type_de_champ_spec.rb
+++ b/spec/models/type_de_champ_spec.rb
@@ -462,6 +462,6 @@ describe TypeDeChamp do
   describe '#humanized_conditionable_types_by_category' do
     subject { TypeDeChamp.humanized_conditionable_types_by_category }
 
-    it { is_expected.to eq([["« Oui/Non »", "« Case à cocher seule »", "« Choix simple »", "« Choix multiple »"], ["« Nombre entier »", "« Nombre décimal »"], ["« Communes »", "« EPCI »", "« Départements »", "« Régions »", "« Adresse »", "« Pays »"]]) }
+    it { is_expected.to eq([["« Oui/Non »", "« Case à cocher seule »", "« Choix simple »", "« Choix multiple »"], ["« Nombre entier »", "« Nombre décimal »"], ["« Adresse »", "« Communes »", "« EPCI »", "« Départements »", "« Régions »", "« Pays »"]]) }
   end
 end
diff --git a/spec/system/routing/rules_full_scenario_spec.rb b/spec/system/routing/rules_full_scenario_spec.rb
index 094a284a7..682f7ed10 100644
--- a/spec/system/routing/rules_full_scenario_spec.rb
+++ b/spec/system/routing/rules_full_scenario_spec.rb
@@ -18,13 +18,13 @@ describe 'The routing with rules', js: true do
     procedure.defaut_groupe_instructeur.instructeurs << administrateur.instructeur
   end
 
-  scenario 'Routage à partir d’un champ' do
+  scenario 'Configuration automatique du routage' do
     steps_to_routing_configuration
 
-    choose('À partir d’un champ', allow_label_click: true)
+    choose('Automatique', allow_label_click: true)
     click_on 'Continuer'
 
-    expect(page).to have_text('Routage à partir d’un champ')
+    expect(page).to have_text('Configuration automatique')
 
     choose('Spécialité', allow_label_click: true)
     click_on 'Créer les groupes'
@@ -46,10 +46,10 @@ describe 'The routing with rules', js: true do
     expect(page).to have_select("groupe_instructeur[condition_form][rows][][value]", selected: "scientifique")
   end
 
-  scenario 'Routage avancé' do
+  scenario 'Configuration manuelle du routage' do
     steps_to_routing_configuration
 
-    choose('Avancé', allow_label_click: true)
+    choose('Manuelle', allow_label_click: true)
     click_on 'Continuer'
 
     expect(page).to have_text('Gestion des groupes')
@@ -333,6 +333,6 @@ describe 'The routing with rules', js: true do
     click_on 'Options'
     expect(page).to have_text('Options concernant l’instruction')
     click_on 'Configurer le routage'
-    expect(page).to have_text('Choix du type de routage')
+    expect(page).to have_text('Choix du type de configuration')
   end
 end