2019-07-25-01 (#4140)

2019-07-25-01

app/assets/stylesheets/new_design/helpers.scss

@@ -7,3 +7,7 @@
 .mr-1 {
   margin-right: $default-spacer;
 }
+
+.mb-4 {
+  margin-bottom: 4 * $default-spacer;
+}

app/controllers/invites_controller.rb

@@ -12,6 +12,7 @@ class InvitesController < ApplicationController
       dossier: dossier,
       user: User.find_by(email: email),
       email: email,
+      message: params[:invite_message],
       email_sender: current_user.email
     )
 

app/controllers/new_administrateur/mail_templates_controller.rb

@@ -3,12 +3,14 @@ module NewAdministrateur
     include ActionView::Helpers::SanitizeHelper
 
     def preview
-      @procedure = procedure
+      mail_template = find_mail_template_by_slug(params[:id])
+      dossier = Dossier.new(id: '1', procedure: procedure)
+
+      @dossier = dossier
       @logo_url = procedure.logo.url
       @service = procedure.service
-
-      mail_template = find_mail_template_by_slug(params[:id])
       @rendered_template = sanitize(mail_template.body)
+      @actions = mail_template.actions_for_dossier(dossier)
 
       render(template: 'notification_mailer/send_notification', layout: 'mailers/notifications_layout')
     end
@@ -16,16 +18,16 @@ module NewAdministrateur
     private
 
     def procedure
-      @procedure = current_administrateur.procedures.find(params[:procedure_id])
+      @procedure ||= current_administrateur.procedures.find(params[:procedure_id])
     end
 
     def mail_templates
       [
-        @procedure.initiated_mail_template,
+        procedure.initiated_mail_template,
-        @procedure.received_mail_template,
+        procedure.received_mail_template,
-        @procedure.closed_mail_template,
+        procedure.closed_mail_template,
-        @procedure.refused_mail_template,
+        procedure.refused_mail_template,
-        @procedure.without_continuation_mail_template
+        procedure.without_continuation_mail_template
       ]
     end
 

app/helpers/mailer_helper.rb

@@ -1,5 +1,28 @@
 module MailerHelper
-  def round_button(text, url)
+  def vertical_margin(height)
-    render 'shared/mailer_round_button', text: text, url: url
+    render 'shared/mailer_vertical_margin', height: height
+  end
+
+  def round_button(text, url, variant)
+    render 'shared/mailer_round_button', text: text, url: url, theme: theme(variant)
+  end
+
+  private
+
+  def theme(variant)
+    case variant
+    when :primary
+      { color: white, bg_color: blue, border_color: blue }
+    when :secondary
+      { color: blue, bg_color: white, border_color: blue }
+    end
+  end
+
+  def blue
+    '#0069CC'
+  end
+
+  def white
+    '#FFFFFF'
   end
 end

app/mailers/notification_mailer.rb

@@ -9,6 +9,7 @@ class NotificationMailer < ApplicationMailer
   include ActionView::Helpers::SanitizeHelper
 
   helper ServiceHelper
+  helper MailerHelper
 
   layout 'mailers/notifications_layout'
 
@@ -46,6 +47,7 @@ class NotificationMailer < ApplicationMailer
     @service = dossier.procedure.service
     @logo_url = attach_logo(dossier.procedure)
     @rendered_template = sanitize(body)
+    @actions = mail_template.actions_for_dossier(dossier)
 
     mail(subject: subject, to: email, template_name: 'send_notification')
   end

app/models/concerns/mail_template_concern.rb

@@ -3,6 +3,12 @@ module MailTemplateConcern
 
   include TagsSubstitutionConcern
 
+  module Actions
+    SHOW         = :show
+    ASK_QUESTION = :ask_question
+    REPLY        = :reply
+  end
+
   def subject_for_dossier(dossier)
     replace_tags(subject, dossier)
   end
@@ -11,6 +17,10 @@ module MailTemplateConcern
     replace_tags(body, dossier)
   end
 
+  def actions_for_dossier(dossier)
+    [MailTemplateConcern::Actions::SHOW, MailTemplateConcern::Actions::ASK_QUESTION]
+  end
+
   def update_rich_body
     self.rich_body = self.body
   end

app/models/mails/initiated_mail.rb

@@ -7,7 +7,7 @@ module Mails
     SLUG = "initiated_mail"
     DEFAULT_TEMPLATE_NAME = "notification_mailer/default_templates/initiated_mail"
     DISPLAYED_NAME = 'Accusé de réception'
-    DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- a bien été reçu (--libellé démarche--)'
+    DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- a bien été déposé (--libellé démarche--)'
     DOSSIER_STATE = Dossier.states.fetch(:en_construction)
   end
 end

app/models/mails/received_mail.rb

@@ -7,7 +7,7 @@ module Mails
     SLUG = "received_mail"
     DEFAULT_TEMPLATE_NAME = "notification_mailer/default_templates/received_mail"
     DISPLAYED_NAME = 'Accusé de passage en instruction'
-    DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- va être instruit (--libellé démarche--)'
+    DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- va être examiné (--libellé démarche--)'
     DOSSIER_STATE = Dossier.states.fetch(:en_instruction)
   end
 end

app/models/mails/refused_mail.rb

@@ -9,5 +9,9 @@ module Mails
     DISPLAYED_NAME = 'Accusé de rejet du dossier'
     DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- a été refusé (--libellé démarche--)'
     DOSSIER_STATE = Dossier.states.fetch(:refuse)
+
+    def actions_for_dossier(dossier)
+      [MailTemplateConcern::Actions::REPLY, MailTemplateConcern::Actions::SHOW]
+    end
   end
 end

app/views/dossier_mailer/notify_new_answer.html.haml

@@ -5,13 +5,14 @@
   Bonjour,
 
 %p
-  L’administration en charge de votre dossier vous a
+  Vous avez reçu un
-  %strong envoyé un nouveau message.
+  %strong nouveau message
+  de la part du service en charge de votre dossier.
 
 %p
-  Pour le consulter et y répondre, cliquez sur le bouton ci-dessous :
+  Pour consulter le message et y répondre, cliquez sur le bouton ci-dessous :
 
-= round_button('Lire le message', messagerie_dossier_url(@dossier))
+= round_button('Lire le message', messagerie_dossier_url(@dossier), :primary)
 
 = render 'layouts/mailers/signature', service: @service
 

app/views/dossier_mailer/notify_new_draft.html.haml

@@ -5,13 +5,16 @@
   Bonjour,
 
 %p
-  Vous avez commencé à remplir un dossier pour la démarche « #{@dossier.procedure.libelle} ».
+  Vous avez commencé à remplir un dossier pour la démarche
+  = succeed '.' do
+    %strong « #{@dossier.procedure.libelle} »
 
 %p
   Vous pouvez
   %strong retrouver et compléter votre dossier
-  à l’adresse suivante :
+  en cliquant sur le bouton ci-dessous:
-  = link_to dossier_url(@dossier), dossier_url(@dossier), target: '_blank', rel: 'noopener'
+
+= round_button('Afficher votre dossier', dossier_url(@dossier), :primary)
 
 = render 'layouts/mailers/signature'
 

app/views/invite_mailer/invite_guest.html.haml

@@ -8,6 +8,11 @@
   %strong= @invite.dossier.procedure.libelle
   sur demarches-simplifiees.fr.
 
+- if @invite.message.present?
+  %blockquote
+    %p
+      = @invite.message
+
 %p
   Cette plateforme permet à ses utilisateurs d'établir des dossiers 100 % en ligne et de dialoguer avec plusieurs interlocuteurs privilégiés avant d'instruire un dépot.
 

app/views/invite_mailer/invite_user.html.haml

@@ -6,6 +6,11 @@
   = @invite.email_sender
   souhaite que vous participiez à l'élaboration d'un dossier sur demarches-simplifiees.fr.
 
+- if @invite.message.present?
+  %blockquote
+    %p
+      = @invite.message
+
 %p
   Pour le consulter, merci de suivre ce lien :
   - url_for_link = invite_url(@invite)

app/views/invites/_dropdown.html.haml

@@ -1,4 +1,4 @@
-%span.dropdown.invite-user-action
+%span.dropdown.invite-user-action{ :style => "float:right;right: 10px;" }
   %button.button.dropdown-button
     %span.icon.person
     - if dossier.invites.count > 0

app/views/invites/_form.html.haml

@@ -13,5 +13,10 @@
     %p Cette personne aura le droit de modifier votre dossier.
 
   = form_tag dossier_invites_path(dossier), remote: true, method: :post, class: 'form' do
-    = email_field_tag :invite_email, '', class: 'small', placeholder: 'adresse email', required: true
+    .row
-    = submit_tag 'Envoyer une invitation', class: 'button accepted'
+      .col
+        = email_field_tag :invite_email, '', class: 'small', placeholder: 'adresse email', required: true
+      .col
+        = text_area_tag :invite_message, '', class: 'small', placeholder: 'Ajouter un message à la personne invitée (optionnel)'
+      .col
+        = submit_tag 'Envoyer une invitation', class: 'button accepted'

app/views/layouts/mailers/layout.html.erb

@@ -9,7 +9,7 @@
       <!--<![endif]-->
       <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
       <meta name="viewport" content="width=device-width, initial-scale=1.0">
-      <style type="text/css">  #outlook a { padding: 0; }  .ReadMsgBody { width: 100%; }  .ExternalClass { width: 100%; }  .ExternalClass * { line-height:100%; }  body { margin: 0; padding: 0; -webkit-text-size-adjust: 100%; -ms-text-size-adjust: 100%; }  table, td { border-collapse:collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; }  img { border: 0; height: auto; line-height: 100%; outline: none; text-decoration: none; -ms-interpolation-mode: bicubic; }  p { display: block; margin: 13px 0; }</style>
+      <style type="text/css">  #outlook a { padding: 0; }  .ReadMsgBody { width: 100%; }  .ExternalClass { width: 100%; }  .ExternalClass * { line-height:100%; }  body { margin: 0; padding: 0; -webkit-text-size-adjust: 100%; -ms-text-size-adjust: 100%; }  table, td { border-collapse:collapse; mso-table-lspace: 0pt; mso-table-rspace: 0pt; }  img { border: 0; height: auto; line-height: 100%; outline: none; text-decoration: none; -ms-interpolation-mode: bicubic; }  p { display: block; margin: 13px 0; } blockquote { margin: 0; } blockquote p { padding: 15px; background: #eee; border-radius: 5px; } blockquote p::before { content: '\201C'; } blockquote p::after { content: '\201D'; }</style>
       <!--[if !mso]>
         <!-->
           <style type="text/css">  @media only screen and (max-width:480px) {    @-ms-viewport { width:320px; }    @viewport { width:320px; }  }</style>

app/views/notification_mailer/_actions.html.haml

@@ -0,0 +1,13 @@
+= vertical_margin(12)
+
+- actions.each_with_index do |action, index|
+  - variant = (index == 0 ? :primary : :secondary)
+  - case action
+  - when MailTemplateConcern::Actions::SHOW
+    = round_button('Consulter mon dossier', dossier_url(@dossier), variant)
+  - when MailTemplateConcern::Actions::ASK_QUESTION
+    = round_button('J’ai une question', messagerie_dossier_url(@dossier), variant)
+  - when MailTemplateConcern::Actions::REPLY
+    = round_button('Répondre à ce message', messagerie_dossier_url(@dossier), variant)
+
+= vertical_margin(5)

app/views/notification_mailer/default_templates/closed_mail.html.haml

@@ -2,7 +2,9 @@
   Bonjour,
 
 %p
-  Votre dossier nº --numéro du dossier-- a été accepté le --date de décision--.
+  Votre dossier nº --numéro du dossier--
+  %strong a été accepté
+  le --date de décision--.
 
 %p
   À tout moment, vous pouvez consulter votre dossier et les éventuels messages de l'administration à cette adresse : --lien dossier--

app/views/notification_mailer/default_templates/closed_mail_with_attestation.html.haml

@@ -2,12 +2,11 @@
   Bonjour,
 
 %p
-  Votre dossier nº --numéro du dossier-- a été accepté le --date de décision--.
+  Votre dossier nº --numéro du dossier--
+  %strong a été accepté
+  le --date de décision--.
 
 %p
   Vous pouvez télécharger votre attestation à l'adresse suivante : --lien attestation--
 
-%p
-  À tout moment, vous pouvez consulter votre dossier et les éventuels messages de l'administration à cette adresse : --lien dossier--
-
 = render partial: "notification_mailer/default_templates/signature"

app/views/notification_mailer/default_templates/initiated_mail.html.haml

@@ -2,9 +2,9 @@
   Bonjour,
 
 %p
-  Votre administration vous confirme la bonne réception de votre dossier nº --numéro du dossier--.
+  Votre dossier nº --numéro du dossier--
-
+  = succeed '.' do
-%p
+    %strong a bien été déposé
-  À tout moment, vous pouvez consulter votre dossier et les éventuels messages de l'administration à cette adresse : --lien dossier--
+  Si besoin est, vous pouvez encore y apporter des modifications.
 
 = render partial: "notification_mailer/default_templates/signature"

app/views/notification_mailer/default_templates/received_mail.html.haml

@@ -2,6 +2,10 @@
   Bonjour,
 
 %p
-  Votre administration vous confirme la bonne réception de votre dossier nº --numéro du dossier--. Celui-ci sera instruit dans le délai légal déclaré par votre interlocuteur.
+  Votre dossier nº --numéro du dossier--
+  a bien été reçu et
+  = succeed '.' do
+    %strong pris en charge
+  Il va maintenant être examiné par le service.
 
 = render partial: "notification_mailer/default_templates/signature"

app/views/notification_mailer/default_templates/refused_mail.html.haml

@@ -2,10 +2,12 @@
   Bonjour,
 
 %p
-  Votre dossier nº --numéro du dossier-- a été refusé le --date de décision--.
+  Votre dossier nº --numéro du dossier--
+  %strong a été refusé
+  le --date de décision--.
 
 %p
-  Le motif de refus est le suivant : --motivation--.
+  Le motif de refus est le suivant : « <i>--motivation--</i> ».
 
 %p
   Pour en savoir plus sur le motif du refus, vous pouvez consulter votre dossier et les éventuels messages de l'administration à cette adresse : --lien dossier--

app/views/notification_mailer/default_templates/without_continuation_mail.html.haml

@@ -2,7 +2,9 @@
   Bonjour,
 
 %p
-  Votre dossier nº --numéro du dossier-- a été classé sans suite le --date de décision--.
+  Votre dossier nº --numéro du dossier--
+  %strong a été classé sans suite
+  le --date de décision--.
 
 %p
   Le motif est le suivant : --motivation--.

app/views/notification_mailer/send_notification.html.haml

@@ -3,5 +3,8 @@
 
 = @rendered_template
 
+- if @actions.present?
+  = render 'notification_mailer/actions', actions: @actions, dossier: @dossier
+
 - content_for :footer do
   = render 'layouts/mailers/service_footer', service: @service, dossier: @dossier

app/views/shared/_mailer_round_button.html.haml

@@ -1,9 +1,10 @@
 /# From https://litmus.com/blog/a-guide-to-bulletproof-buttons-in-email-design
-%table{ width: "100%", border: "0", cellspacing:"0", cellpadding:"0" }
+%table{ width: "100%", border: "0", cellspacing:"0", cellpadding:"5" }
   %tr
-    %td
+    %td{ align: "center" }
-      %table{ border:"0", cellspacing:"0", cellpadding:"0", style:"margin: auto" }
+      %table{ border:"0", cellspacing:"0", cellpadding:"0" }
         %tr
-          %td{ align:"center", style:"border-radius: 5px;", bgcolor:"#0069cc" }
+          %td{ align:"center", style:"border-radius: 5px;", color: theme[:color], bgcolor: theme[:bg_color] }
-            %a{ href: url, target:"_blank", style:"font-size: 16px; font-family: Helvetica, Arial, sans-serif; color: #ffffff; text-decoration: none; text-decoration: none; border-radius: 5px; padding: 12px 25px; border: 1px solid #0069cc; display: inline-block;" }
+            %a{ href: url, target:"_blank", rel: "noopener", style:"font-size: 16px; font-family: Helvetica, Arial, sans-serif; color: #{theme[:color]}; text-decoration: none; text-decoration: none; border-radius: 5px; padding: 12px 25px; border: 1px solid #{theme[:border_color]}; display: inline-block; min-width: 250px" }
               = text
+

app/views/shared/_mailer_vertical_margin.html.haml

@@ -0,0 +1,4 @@
+%table{ cellspacing: "0", cellpadding: (height / 2), border: "0" }
+  %tr
+    %td
+      %div

app/views/users/dossiers/_dossier_actions.html.haml

@@ -9,6 +9,20 @@
       Actions
     .dropdown-content.fade-in-down
       %ul.dropdown-items
+        - if !dossier.read_only?
+          - if dossier.brouillon?
+            %li
+              = link_to(url_for_dossier(dossier)) do
+                %span.icon.edit
+                .dropdown-description
+                  Modifier le brouillon
+          - else
+            %li
+              = link_to modifier_dossier_path(dossier) do
+                %span.icon.edit
+                .dropdown-description
+                  Modifier le dossier
+
         - if has_new_dossier_action
           %li
             = link_to procedure_lien(dossier.procedure) do

app/views/users/dossiers/demande.html.haml

@@ -10,5 +10,5 @@
 
   .container
     - if !@dossier.read_only?
-      = link_to "Modifier le dossier", modifier_dossier_path(@dossier), class: 'button primary edit-form'
+      = link_to "Modifier le dossier", modifier_dossier_path(@dossier), class: 'button accepted edit-form', 'title'=> "Vous pouvez modifier votre dossier tant qu'il n'est passé en instruction"
       .clearfix

app/views/users/dossiers/show/_header.html.haml

@@ -11,8 +11,9 @@
           = "- Déposé le #{l(dossier.en_construction_at, format: '%d %B %Y')}"
 
     - if current_user.owns?(dossier)
-      .header-actions
+      = link_to "Modifier mon dossier", modifier_dossier_path(dossier.id), class: 'button accepted edit-form', 'title'=> "Vous pouvez modifier votre dossier tant qu'il n'est passé en instruction"
-        = render partial: 'invites/dropdown', locals: { dossier: dossier }
+      = render partial: 'invites/dropdown', locals: { dossier: dossier }
+      .clearfix
 
     %ul.tabs
       = dynamic_tab_item('Résumé', dossier_path(dossier))

app/views/users/dossiers/siret.html.haml

@@ -8,7 +8,12 @@
 
     = f.label :siret, style: "display: none"
     = f.text_field :siret, placeholder: "Numéro SIRET à 14 chiffres", required: true
-
     = f.hidden_field :dossier_id, value: @dossier.id
 
-    = f.submit "Valider", class: "button large primary expand", data: { disable_with: "Récupération des informations…" }
+    %p.mb-4
+      Pour trouver votre numéro SIRET, utilisez
+      %a{ href: 'https://entreprise.data.gouv.fr/', target: '_blank', rel: 'noopener' }
+        entreprise.data.gouv.fr
+      ou renseignez-vous auprès de votre service comptable.
+
+    = f.submit "Valider", class: "button large primary expand mt-1", data: { disable_with: "Récupération des informations…" }

db/migrate/20190717151228_add_message_to_invites.rb

@@ -0,0 +1,5 @@
+class AddMessageToInvites < ActiveRecord::Migration[5.2]
+  def change
+    add_column :invites, :message, :text
+  end
+end

spec/controllers/new_administrateur/mail_templates_controller_spec.rb

@@ -13,9 +13,17 @@ describe NewAdministrateur::MailTemplatesController, type: :controller do
 
     it { expect(response).to have_http_status(:ok) }
 
-    it { expect(response.body).to have_css("img[src*='#{procedure.logo.filename}']") }
+    it 'displays the procedure logo' do
+      expect(response.body).to have_css("img[src*='#{procedure.logo.filename}']")
+    end
 
-    it { expect(response.body).to include(procedure.service.nom) }
+    it 'displays the action buttons' do
-    it { expect(response.body).to include(procedure.service.telephone) }
+      expect(response.body).to have_link('Consulter mon dossier')
+    end
+
+    it 'displays the service in the footer' do
+      expect(response.body).to include(procedure.service.nom)
+      expect(response.body).to include(procedure.service.telephone)
+    end
   end
 end

spec/mailers/notification_mailer_spec.rb

@@ -28,6 +28,11 @@ RSpec.describe NotificationMailer, type: :mailer do
       expect(mail.body).to have_link('messagerie')
     end
 
+    it 'renders the actions' do
+      expect(mail.body).to have_link('Consulter mon dossier', href: dossier_url(dossier))
+      expect(mail.body).to have_link('J’ai une question', href: messagerie_dossier_url(dossier))
+    end
+
     context 'when the template body contains tags' do
       let(:email_template) { create(:received_mail, subject: 'Email subject', body: 'Hello --nom--, your dossier --lien dossier-- was processed.') }
 

spec/mailers/previews/notification_mailer_preview.rb

@@ -1,19 +1,20 @@
 class NotificationMailerPreview < ActionMailer::Preview
-  def send_dossier_received
-    NotificationMailer.send_dossier_received(Dossier.last)
-  end
-
   def send_initiated_notification
     p = Procedure.where(id: Mails::InitiatedMail.where("body like ?", "%<img%").pluck(:procedure_id).uniq).order("RANDOM()").first
     NotificationMailer.send_initiated_notification(p.dossiers.last)
   end
 
+  def send_dossier_received
+    NotificationMailer.send_dossier_received(Dossier.last)
+  end
+
   def send_closed_notification
     NotificationMailer.send_closed_notification(Dossier.last)
   end
 
   def send_refused_notification
-    NotificationMailer.send_refused_notification(Dossier.last)
+    dossier = Dossier.last.tap { |d| d.assign_attributes(motivation: 'Le montant demandé dépasse le plafond autorisé') }
+    NotificationMailer.send_refused_notification(dossier)
   end
 
   def send_without_continuation_notification