From f768cb2dae72f4ff1819fff7fbcb0617acaaf6e9 Mon Sep 17 00:00:00 2001
From: Mathieu Magnin <mathieu.magnin@gmail.com>
Date: Thu, 4 Apr 2019 12:06:28 +0200
Subject: [PATCH 1/2] [Fix #3721] Change notification screen to make it
 understandable

---
 app/assets/stylesheets/new_design/forms.scss  | 12 +++++++++
 .../procedures/email_notifications.html.haml  | 26 ++++++++++++++++++-
 2 files changed, 37 insertions(+), 1 deletion(-)

diff --git a/app/assets/stylesheets/new_design/forms.scss b/app/assets/stylesheets/new_design/forms.scss
index c260ec713..00fc6984c 100644
--- a/app/assets/stylesheets/new_design/forms.scss
+++ b/app/assets/stylesheets/new_design/forms.scss
@@ -92,6 +92,18 @@
         margin-left: 0;
       }
     }
+
+    &.vertical {
+      label {
+        display: block;
+        margin-left: 0;
+        margin-bottom: 0;
+      }
+
+      input[type=radio] {
+        margin-bottom: 16px;
+      }
+    }
   }
 
   input[type=text]:not([data-address='true']),
diff --git a/app/views/gestionnaires/procedures/email_notifications.html.haml b/app/views/gestionnaires/procedures/email_notifications.html.haml
index 751be27bf..72a993a35 100644
--- a/app/views/gestionnaires/procedures/email_notifications.html.haml
+++ b/app/views/gestionnaires/procedures/email_notifications.html.haml
@@ -8,7 +8,16 @@
   %h1 Notifications par email
 
   = form_for @assign_to, url: update_email_notifications_gestionnaire_procedure_path(@procedure), html: { class: 'form' } do |form|
-    = form.label :email_notification, "Recevoir une fois par jour, du lundi au samedi vers 10 h, un email de notification me signalant le dépôt de nouveaux dossiers ou des changements sur mes dossiers en cours"
+    .explication
+      Configurez sur cette page les notifications que vous souhaitez recevoir par email pour cette démarche.
+
+    = form.label :email_notification do
+      Recevoir une notification quotidienne
+      
+      %span.notice
+        Cet email vous signale le dépôt de nouveaux dossiers sur cette démarche, ou des changements sur vos dossiers suivis.
+      %span.notice
+        Il est envoyé une fois par jour, du lundi au samedi, vers 10 h du matin.
 
     .radios
       %label
@@ -19,6 +28,21 @@
         = form.radio_button :email_notifications_enabled, false
         Non
 
+    = form.label nil do
+      Recevoir un récapitulatif hebdomadaire
+      %span.notice
+        Cet email récapitule l’activité de la semaine sur l’ensemble de vos démarches.
+      %span.notice
+        Il est envoyé chaque semaine, et n’est pas désactivable.
+    
+    .radios
+      %label
+        = radio_button_tag "not_implemented", "Oui", true, disabled: true
+        Oui
+      %label
+        = radio_button_tag "not_implemented", "Non", false, disabled: true
+        Non
+
     .send-wrapper
       = link_to "Revenir à la procédure", gestionnaire_procedure_path(@procedure), class: 'button mr-1'
       = form.submit "Enregistrer", class: "button primary"

From 3de9148e33ed0a73e68f3df87741c14f5286fb07 Mon Sep 17 00:00:00 2001
From: Mathieu Magnin <mathieu.magnin@gmail.com>
Date: Thu, 4 Apr 2019 14:54:38 +0200
Subject: [PATCH 2/2] Change WeeklyOverviewJob hour and day

---
 README.md                                                   | 2 +-
 .../gestionnaires/procedures/email_notifications.html.haml  | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/README.md b/README.md
index a84994dba..39f802e98 100644
--- a/README.md
+++ b/README.md
@@ -62,7 +62,7 @@ En local, un utilisateur de test est créé automatiquement, avec les identifian
 ### Programmation des jobs
 
     AutoArchiveProcedureJob.set(cron: "* * * * *").perform_later
-    WeeklyOverviewJob.set(cron: "0 8 * * 0").perform_later
+    WeeklyOverviewJob.set(cron: "0 7 * * 1").perform_later
     AutoReceiveDossiersForProcedureJob.set(cron: "* * * * *").perform_later(procedure_declaratoire_id, Dossier.states.fetch(:en_instruction))
     SendinblueUpdateAdministrateursJob.set(cron: "0 10 * * *").perform_later
     FindDubiousProceduresJob.set(cron: "0 0 * * *").perform_later
diff --git a/app/views/gestionnaires/procedures/email_notifications.html.haml b/app/views/gestionnaires/procedures/email_notifications.html.haml
index 72a993a35..a061b5d38 100644
--- a/app/views/gestionnaires/procedures/email_notifications.html.haml
+++ b/app/views/gestionnaires/procedures/email_notifications.html.haml
@@ -13,7 +13,7 @@
 
     = form.label :email_notification do
       Recevoir une notification quotidienne
-      
+
       %span.notice
         Cet email vous signale le dépôt de nouveaux dossiers sur cette démarche, ou des changements sur vos dossiers suivis.
       %span.notice
@@ -33,8 +33,8 @@
       %span.notice
         Cet email récapitule l’activité de la semaine sur l’ensemble de vos démarches.
       %span.notice
-        Il est envoyé chaque semaine, et n’est pas désactivable.
-    
+        Il est envoyé chaque semaine le lundi matin, et n’est pas désactivable.
+
     .radios
       %label
         = radio_button_tag "not_implemented", "Oui", true, disabled: true