Merge branch 'dev'

app/assets/stylesheets/new_design/demarches_index.scss

@@ -30,3 +30,7 @@
     background-color: $lighter-blue;
   }
 }
+
+.service-name {
+  color: $grey;
+}

app/controllers/new_user/demarches_controller.rb

@@ -9,6 +9,7 @@ module NewUser
         .select { |p| p.publiee? }
 
       @popular_demarches = Procedure
+        .includes(:service)
         .select("procedures.*, COUNT(*) AS procedures_count")
         .joins(:dossiers)
         .publiees

app/views/new_user/demarches/index.html.haml

@@ -12,14 +12,22 @@
       %ul.demarche-links
         - @previous_demarches_still_active.each do |demarche|
           %li
-            = link_to(demarche.libelle, commencer_url(procedure_path: demarche.path), class: "demarche-link")
+            = link_to(commencer_url(procedure_path: demarche.path), class: "demarche-link") do
+              = demarche.libelle
+              %br
+              .service-name
+                = demarche&.service&.nom
 
     - if @popular_demarches.present?
       %h2.list-header LES PLUS POPULAIRES
       %ul.demarche-links
         - @popular_demarches.each do |demarche|
           %li
-            = link_to(demarche.libelle, commencer_url(procedure_path: demarche.path), class: "demarche-link")
+            = link_to(commencer_url(procedure_path: demarche.path), class: "demarche-link") do
+              = demarche.libelle
+              %br
+              .service-name
+                = demarche&.service&.nom
 
     %h2.list-header TOUTES LES DÉMARCHES
     = link_to("Voir l'intégralité des démarches disponibles", LISTE_DES_DEMARCHES_URL, class: "button")

app/views/new_user/dossiers/show/_status_overview.html.haml

@@ -1,8 +1,9 @@
 .status-overview
   - if !dossier.termine?
     %ul.status-timeline
-      %li.brouillon{ class: dossier.brouillon? ? 'active' : nil }
+      - if dossier.brouillon?
-        brouillon
+        %li.brouillon{ class: dossier.brouillon? ? 'active' : nil }
+          brouillon
       %li.en-construction{ class: dossier.en_construction? ? 'active' : nil }
         en construction
       %li.en-instruction{ class: dossier.en_instruction? ? 'active' : nil }
@@ -18,13 +19,21 @@
 
     - elsif dossier.en_construction?
       .en-construction
-        %p Un accompagnant de l’administration est en train de vérifier que votre dossier est bien complet.
+        %p Un instructeur de l’administration est en train de vérifier que votre dossier est bien complet. Si des modifications sont nécessaires, vous recevrez un message avec les modifications à effectuer.
-        %p Si des modifications sont nécessaires, vous recevrez un email avec les modifications à effectuer. Et sinon, dès que votre dossier sera complet, il passera automatiquement en instruction.
+
+        %p
+          Sinon,
+          = succeed '.' do
+            %strong votre dossier passera directement en instruction
 
     - elsif dossier.en_instruction?
       .en-instruction
-        %p Votre dossier est complet. Il est en cours d’examen par les agent·e·s de l’administration.
+        %p Votre dossier est complet. Il est en cours d’examen par les instructeur de l’administration.
-        %p Dès que l’administration aura statué sur votre dossier, vous recevrez un email avec le résultat.
+        %p
+          Dès que l’administration aura statué sur votre dossier,
+          %strong
+            vous recevrez un email
+          avec le résultat.
 
     - elsif dossier.accepte?
       .accepte

lib/tasks/2018_09_12_fix_templates.rake

@@ -2,7 +2,7 @@ namespace :'2018_09_12_fix_templates' do
   task run: :environment do
     dossiers_with_invalid_attestations = find_dossiers_with_sent_and_invalid_attestations
     fix_templates
-    fixed_attestations = regenerate_attestations(dossiers_with_invalid_attestations)
+    fixed_attestations = delete_then_regenerate_attestations(dossiers_with_invalid_attestations)
     send_regenerated_attestations(dossiers_with_invalid_attestations)
   end
 

spec/views/new_user/dossiers/show/_status_overview.html.haml_spec.rb

@@ -39,7 +39,7 @@ describe 'new_user/dossiers/show/_status_overview.html.haml', type: :view do
     let(:dossier) { create :dossier, :en_construction }
 
     it 'renders the timeline (without the final states)' do
-      expect(rendered).to have_timeline_item('.brouillon').inactive
+      expect(rendered).not_to have_timeline_item('.brouillon')
       expect(rendered).to have_timeline_item('.en-construction').active
       expect(rendered).to have_timeline_item('.en-instruction').inactive
       expect(rendered).to have_timeline_item('.termine').inactive
@@ -52,7 +52,7 @@ describe 'new_user/dossiers/show/_status_overview.html.haml', type: :view do
     let(:dossier) { create :dossier, :en_instruction }
 
     it 'renders the timeline (without the final states)' do
-      expect(rendered).to have_timeline_item('.brouillon').inactive
+      expect(rendered).not_to have_timeline_item('.brouillon')
       expect(rendered).to have_timeline_item('.en-construction').inactive
       expect(rendered).to have_timeline_item('.en-instruction').active
       expect(rendered).to have_timeline_item('.termine').inactive