From 718a30639ffae4ea04ab23dc2c2449e1da96096e Mon Sep 17 00:00:00 2001 From: Mathieu Magnin Date: Fri, 23 Nov 2018 11:04:28 +0100 Subject: [PATCH] [Fix #2929] Improve readability of invitation for avis --- app/mailers/avis_mailer.rb | 2 ++ app/views/avis_mailer/avis_invitation.html.haml | 13 +++++++++---- spec/mailers/avis_mailer_spec.rb | 2 +- 3 files changed, 12 insertions(+), 5 deletions(-) diff --git a/app/mailers/avis_mailer.rb b/app/mailers/avis_mailer.rb index b4ca982c1..78c7ac5a8 100644 --- a/app/mailers/avis_mailer.rb +++ b/app/mailers/avis_mailer.rb @@ -1,5 +1,7 @@ # Preview all emails at http://localhost:3000/rails/mailers/avis_mailer class AvisMailer < ApplicationMailer + layout 'mailers/layout' + def avis_invitation(avis) @avis = avis email = @avis.gestionnaire&.email || @avis.email diff --git a/app/views/avis_mailer/avis_invitation.html.haml b/app/views/avis_mailer/avis_invitation.html.haml index c5ba3e20b..e339c9a88 100644 --- a/app/views/avis_mailer/avis_invitation.html.haml +++ b/app/views/avis_mailer/avis_invitation.html.haml @@ -1,16 +1,21 @@ +- content_for(:title, 'Vous avez été invité à donner votre avis') + - avis_link = @avis.gestionnaire.present? ? gestionnaire_avis_url(@avis) : sign_up_gestionnaire_avis_url(@avis.id, @avis.email) %p Bonjour, %p - = "Vous avez été invité par #{@avis.claimant.email} à donner votre avis sur le dossier nº #{@avis.dossier.id} de la démarche \"#{@avis.dossier.procedure.libelle}\"." + Vous avez été invité par + %strong= @avis.claimant.email + = "à donner votre avis sur le dossier nº #{@avis.dossier.id} de la démarche :" + %strong= @avis.dossier.procedure.libelle %p - Message de votre interlocuteur : + = "#{@avis.claimant.email} vous a écrit :" %br - %span{ style: 'font-style: italic;' } - = @avis.introduction +%p{ style: "padding: 8px; color: #333333; background-color: #EEEEEE; font-size: 16px;" } + = @avis.introduction - if @avis.gestionnaire.present? %p diff --git a/spec/mailers/avis_mailer_spec.rb b/spec/mailers/avis_mailer_spec.rb index 9b3b07e6e..70f826b13 100644 --- a/spec/mailers/avis_mailer_spec.rb +++ b/spec/mailers/avis_mailer_spec.rb @@ -7,7 +7,7 @@ RSpec.describe AvisMailer, type: :mailer do subject { described_class.avis_invitation(avis) } it { expect(subject.subject).to eq("Donnez votre avis sur le dossier nº #{avis.dossier.id} (#{avis.dossier.procedure.libelle})") } - it { expect(subject.body).to include("Vous avez été invité par #{avis.claimant.email} à donner votre avis sur le dossier nº #{avis.dossier.id} de la démarche "#{avis.dossier.procedure.libelle}".") } + it { expect(subject.body).to have_text("Vous avez été invité par #{avis.claimant.email} à donner votre avis sur le dossier nº #{avis.dossier.id} de la démarche : #{avis.dossier.procedure.libelle}") } it { expect(subject.body).to include(avis.introduction) } it { expect(subject.body).to include(gestionnaire_avis_url(avis)) }