From f768cb2dae72f4ff1819fff7fbcb0617acaaf6e9 Mon Sep 17 00:00:00 2001 From: Mathieu Magnin Date: Thu, 4 Apr 2019 12:06:28 +0200 Subject: [PATCH 1/4] [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 Date: Thu, 4 Apr 2019 14:54:38 +0200 Subject: [PATCH 2/4] 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 From e90b1cdcc0b8c7222702625cd30a85f01ae78802 Mon Sep 17 00:00:00 2001 From: Pierre de La Morinerie Date: Thu, 4 Apr 2019 17:14:32 +0200 Subject: [PATCH 3/4] api_entreprise: use API v3 again This reverts commit 10e1408a882882954bcf911a58bd275d040df94d. --- app/lib/api_entreprise/api.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/lib/api_entreprise/api.rb b/app/lib/api_entreprise/api.rb index 1f0f52fd4..79b2cc766 100644 --- a/app/lib/api_entreprise/api.rb +++ b/app/lib/api_entreprise/api.rb @@ -40,7 +40,9 @@ class ApiEntreprise::API end def self.url(resource_name, siret_or_siren) - [API_ENTREPRISE_URL, resource_name, siret_or_siren].join("/") + base_url = [API_ENTREPRISE_URL, resource_name, siret_or_siren].join("/") + + "#{base_url}?with_insee_v3=true" end def self.params(siret_or_siren, procedure_id) From 01e113d04f2202358821831d0828d5b1f1a73793 Mon Sep 17 00:00:00 2001 From: Pierre de La Morinerie Date: Thu, 4 Apr 2019 17:15:31 +0200 Subject: [PATCH 4/4] Revert "locales: tell that API Entreprise is unavailable for now" This reverts commit a5983838561848a344b7140a4542c9f21879833a. --- config/locales/fr.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/locales/fr.yml b/config/locales/fr.yml index c7aa81890..4000e6ece 100644 --- a/config/locales/fr.yml +++ b/config/locales/fr.yml @@ -182,7 +182,7 @@ fr: dossier_map_not_activated: "Le dossier n'a pas accès à la cartographie." invalid_siret: "Le siret est incorrect" procedure_not_found: "La démarche n'existe pas" - siret_unknown: 'Désolé, la récupération des informations SIRET est temporairement indisponible. Veuillez ré-essayer dans une heure.' + siret_unknown: 'Désolé, nous n’avons pas trouvé d’établissement enregistré correspondant à ce numéro SIRET' etablissement_fail: 'Désolé, nous n’avons pas réussi à enregistrer l’établissement correspondant à ce numéro SIRET' france_connect: connexion: "Erreur lors de la connexion à France Connect."