diff --git a/app/mailers/dossier_mailer.rb b/app/mailers/dossier_mailer.rb index a26caff85..c07fed240 100644 --- a/app/mailers/dossier_mailer.rb +++ b/app/mailers/dossier_mailer.rb @@ -4,14 +4,14 @@ class DossierMailer < ApplicationMailer def notify_new_draft(dossier) @dossier = dossier - subject = "Retrouvez votre brouillon pour la démarche \"#{dossier.procedure.libelle}\"" + subject = "Retrouvez votre brouillon pour la démarche « #{dossier.procedure.libelle} »" mail(to: dossier.user.email, subject: subject) end def notify_new_answer(dossier) @dossier = dossier - subject = "Nouveau message pour votre dossier nº #{dossier.id}" + subject = "Nouveau message pour votre dossier nº #{dossier.id} (#{dossier.procedure.libelle})" mail(to: dossier.user.email, subject: subject) do |format| format.html { render layout: 'mailers/notification' } @@ -20,21 +20,21 @@ class DossierMailer < ApplicationMailer def notify_deletion_to_user(deleted_dossier, to_email) @deleted_dossier = deleted_dossier - subject = "Votre dossier n° #{@deleted_dossier.dossier_id} a bien été supprimé" + subject = "Votre dossier nº #{@deleted_dossier.dossier_id} a bien été supprimé" mail(to: to_email, subject: subject) end def notify_deletion_to_administration(deleted_dossier, to_email) @deleted_dossier = deleted_dossier - subject = "Le dossier n° #{@deleted_dossier.dossier_id} a été supprimé à la demande de l'usager" + subject = "Le dossier nº #{@deleted_dossier.dossier_id} a été supprimé à la demande de l'usager" mail(to: to_email, subject: subject) end def notify_unhide_to_user(dossier) @dossier = dossier - subject = "Votre dossier n° #{@dossier.id} n'a pas pu être supprimé" + subject = "Votre dossier nº #{@dossier.id} n'a pas pu être supprimé" mail(to: dossier.user.email, subject: subject) end @@ -42,23 +42,14 @@ class DossierMailer < ApplicationMailer def notify_undelete_to_user(dossier) @dossier = dossier @dossier_kind = dossier.brouillon? ? 'brouillon' : 'dossier' - @subject = "Votre #{@dossier_kind} n° #{@dossier.id} est à nouveau accessible" - - mail(to: dossier.user.email, subject: @subject) - end - - def notify_unmigrated_to_user(dossier, new_procedure) - @dossier = dossier - @dossier_kind = dossier.brouillon? ? 'brouillon' : 'dossier' - @subject = "Changement de procédure pour votre #{@dossier_kind} n° #{@dossier.id}" - @new_procedure = new_procedure + @subject = "Votre #{@dossier_kind} nº #{@dossier.id} est à nouveau accessible" mail(to: dossier.user.email, subject: @subject) end def notify_revert_to_instruction(dossier) @dossier = dossier - @subject = "Votre dossier n° #{@dossier.id} est en train d'être réexaminé" + @subject = "Votre dossier nº #{@dossier.id} est en train d'être réexaminé" mail(to: dossier.user.email, subject: @subject) end diff --git a/app/models/mails/closed_mail.rb b/app/models/mails/closed_mail.rb index 1c954c9f0..a0245bac6 100644 --- a/app/models/mails/closed_mail.rb +++ b/app/models/mails/closed_mail.rb @@ -6,7 +6,7 @@ module Mails SLUG = "closed_mail" DISPLAYED_NAME = "Accusé d'acceptation" - DEFAULT_SUBJECT = 'Votre dossier demarches-simplifiees.fr nº --numéro du dossier-- a été accepté' + DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- a été accepté (--libellé démarche--)' DOSSIER_STATE = Dossier.states.fetch(:accepte) def self.default_template_name_for_procedure(procedure) diff --git a/app/models/mails/initiated_mail.rb b/app/models/mails/initiated_mail.rb index ab438e023..fc32ae5da 100644 --- a/app/models/mails/initiated_mail.rb +++ b/app/models/mails/initiated_mail.rb @@ -7,7 +7,7 @@ module Mails SLUG = "initiated_mail" DEFAULT_TEMPLATE_NAME = "notification_mailer/initiated_mail" DISPLAYED_NAME = 'Accusé de réception' - DEFAULT_SUBJECT = 'Votre dossier demarches-simplifiees.fr nº --numéro du dossier-- a bien été reçu' + DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- a bien été reçu (--libellé démarche--)' DOSSIER_STATE = Dossier.states.fetch(:en_construction) end end diff --git a/app/models/mails/received_mail.rb b/app/models/mails/received_mail.rb index 59832b9dd..5cb3258e1 100644 --- a/app/models/mails/received_mail.rb +++ b/app/models/mails/received_mail.rb @@ -7,7 +7,7 @@ module Mails SLUG = "received_mail" DEFAULT_TEMPLATE_NAME = "notification_mailer/received_mail" DISPLAYED_NAME = 'Accusé de passage en instruction' - DEFAULT_SUBJECT = 'Votre dossier demarches-simplifiees.fr nº --numéro du dossier-- va être instruit' + DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- va être instruit (--libellé démarche--)' DOSSIER_STATE = Dossier.states.fetch(:en_instruction) end end diff --git a/app/models/mails/refused_mail.rb b/app/models/mails/refused_mail.rb index 19d66db10..d4b4d25db 100644 --- a/app/models/mails/refused_mail.rb +++ b/app/models/mails/refused_mail.rb @@ -7,7 +7,7 @@ module Mails SLUG = "refused_mail" DEFAULT_TEMPLATE_NAME = "notification_mailer/refused_mail" DISPLAYED_NAME = 'Accusé de rejet du dossier' - DEFAULT_SUBJECT = 'Votre dossier demarches-simplifiees.fr nº --numéro du dossier-- a été refusé' + DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- a été refusé (--libellé démarche--)' DOSSIER_STATE = Dossier.states.fetch(:refuse) end end diff --git a/app/models/mails/without_continuation_mail.rb b/app/models/mails/without_continuation_mail.rb index b70102b32..11cd9a182 100644 --- a/app/models/mails/without_continuation_mail.rb +++ b/app/models/mails/without_continuation_mail.rb @@ -7,7 +7,7 @@ module Mails SLUG = "without_continuation" DEFAULT_TEMPLATE_NAME = "notification_mailer/without_continuation_mail" DISPLAYED_NAME = 'Accusé de classement sans suite' - DEFAULT_SUBJECT = 'Votre dossier demarches-simplifiees.fr nº --numéro du dossier-- a été classé sans suite' + DEFAULT_SUBJECT = 'Votre dossier nº --numéro du dossier-- a été classé sans suite (--libellé démarche--)' DOSSIER_STATE = Dossier.states.fetch(:sans_suite) end end diff --git a/app/views/dossier_mailer/notify_unmigrated_to_user.html.haml b/app/views/dossier_mailer/notify_unmigrated_to_user.html.haml deleted file mode 100644 index 438f68252..000000000 --- a/app/views/dossier_mailer/notify_unmigrated_to_user.html.haml +++ /dev/null @@ -1,22 +0,0 @@ -- content_for(:title, @subject) - -%h1 Bonjour, - -%p - Vous avez commencé un #{@dossier_kind}, - = link_to("n° #{@dossier.id}", dossier_url(@dossier)) - sur la procédure « #{@dossier.procedure.libelle} ». - En raison d’un changement dans la procédure, votre #{@dossier_kind} a été inaccessible pendant quelques jours. - L’accès est à présent à nouveau possible. - -%p - Malheureusement, en raison des changements dans le procédure, vous ne pourrez pas mener à terme le #{@dossier_kind} commencé. - Si votre démarche est toujours d’actualité, nous vous invitons à la recommencer sur - = link_to("la nouvelle procédure", commencer_url(@new_procedure.path)) - \. - -%p - Nous avons pris des mesures pour nous assurer qu’un tel désagrément ne se reproduise pas, - et vous présentons nos excuses pour la gène occasionnée. - -= render partial: "layouts/mailers/signature" diff --git a/app/views/layouts/mailers/_signature.html.haml b/app/views/layouts/mailers/_signature.html.haml index ea217a250..ca5ad2778 100644 --- a/app/views/layouts/mailers/_signature.html.haml +++ b/app/views/layouts/mailers/_signature.html.haml @@ -1,5 +1,4 @@ %p Bonne journée, - -%p - L'équipe demarches-simplifiees.fr + %br + L’équipe demarches-simplifiees.fr diff --git a/app/views/notification_mailer/_signature.html.haml b/app/views/notification_mailer/_signature.html.haml index 132bfafa4..9aef6f966 100644 --- a/app/views/notification_mailer/_signature.html.haml +++ b/app/views/notification_mailer/_signature.html.haml @@ -1,4 +1,4 @@ %p Bonne journée, - -%p --nom du service-- + %br + \ --nom du service-- diff --git a/spec/factories/commentaire.rb b/spec/factories/commentaire.rb index dc092a50d..360a4b05e 100644 --- a/spec/factories/commentaire.rb +++ b/spec/factories/commentaire.rb @@ -9,7 +9,7 @@ FactoryBot.define do end trait :with_file do - file { Rack::Test::UploadedFile.new("./spec/fixtures/files/logo_test_procedure.png", 'application/pdf') } + file { Rack::Test::UploadedFile.new("./spec/fixtures/files/logo_test_procedure.png", 'image/png') } end end end diff --git a/spec/factories/procedure.rb b/spec/factories/procedure.rb index 8c62169b3..2e5bd901d 100644 --- a/spec/factories/procedure.rb +++ b/spec/factories/procedure.rb @@ -42,7 +42,7 @@ FactoryBot.define do end trait :with_logo do - logo { Rack::Test::UploadedFile.new("./spec/fixtures/files/logo_test_procedure.png", 'application/pdf') } + logo { Rack::Test::UploadedFile.new("./spec/fixtures/files/logo_test_procedure.png", 'image/png') } end trait :with_path do diff --git a/spec/mailers/dossier_mailer_spec.rb b/spec/mailers/dossier_mailer_spec.rb index eaec0fc93..626d1513b 100644 --- a/spec/mailers/dossier_mailer_spec.rb +++ b/spec/mailers/dossier_mailer_spec.rb @@ -29,7 +29,7 @@ RSpec.describe DossierMailer, type: :mailer do subject { described_class.notify_deletion_to_user(deleted_dossier, to_email) } - it { expect(subject.subject).to eq("Votre dossier n° #{deleted_dossier.dossier_id} a bien été supprimé") } + it { expect(subject.subject).to eq("Votre dossier nº #{deleted_dossier.dossier_id} a bien été supprimé") } it { expect(subject.body).to include("Votre dossier") } it { expect(subject.body).to include(deleted_dossier.dossier_id) } it { expect(subject.body).to include("a bien été supprimé") } @@ -41,7 +41,7 @@ RSpec.describe DossierMailer, type: :mailer do subject { described_class.notify_deletion_to_administration(deleted_dossier, to_email) } - it { expect(subject.subject).to eq("Le dossier n° #{deleted_dossier.dossier_id} a été supprimé à la demande de l'usager") } + it { expect(subject.subject).to eq("Le dossier nº #{deleted_dossier.dossier_id} a été supprimé à la demande de l'usager") } it { expect(subject.body).to include("À la demande de l'usager") } it { expect(subject.body).to include(deleted_dossier.dossier_id) } it { expect(subject.body).to include(deleted_dossier.procedure.libelle) } @@ -52,7 +52,7 @@ RSpec.describe DossierMailer, type: :mailer do subject { described_class.notify_unhide_to_user(dossier) } - it { expect(subject.subject).to eq("Votre dossier n° #{dossier.id} n'a pas pu être supprimé") } + it { expect(subject.subject).to eq("Votre dossier nº #{dossier.id} n'a pas pu être supprimé") } it { expect(subject.body).to include(dossier.id) } it { expect(subject.body).to include("n'a pas pu être supprimé") } it { expect(subject.body).to include(dossier.procedure.libelle) } diff --git a/spec/mailers/previews/dossier_mailer_preview.rb b/spec/mailers/previews/dossier_mailer_preview.rb index d34d4f0e6..8ac4eab87 100644 --- a/spec/mailers/previews/dossier_mailer_preview.rb +++ b/spec/mailers/previews/dossier_mailer_preview.rb @@ -1,7 +1,7 @@ # Preview all emails at http://localhost:3000/rails/mailers/dossier_mailer class DossierMailerPreview < ActionMailer::Preview def notify_new_draft - DossierMailer.notify_new_draft(dossier) + DossierMailer.notify_new_draft(draft) end def notify_new_answer @@ -26,11 +26,24 @@ class DossierMailerPreview < ActionMailer::Preview DeletedDossier.new(dossier_id: 1, procedure: procedure) end + def draft + Dossier.new(id: 47882, procedure: procedure, user: User.new(email: "usager@example.com")) + end + def dossier - Dossier.new(id: 1, procedure: procedure, user: User.new(email: "usager@example.com")) + Dossier.new(id: 47882, state: :en_instruction, procedure: procedure, user: User.new(email: "usager@example.com")) end def procedure - Procedure.new(libelle: 'Démarche pour des marches') + Procedure.new(libelle: 'Dotation d’Équipement des Territoires Ruraux - Exercice 2019', service: service, logo: Rack::Test::UploadedFile.new("./spec/fixtures/files/logo_test_procedure.png", 'image/png')) + end + + def service + Service.new( + nom: 'Direction du Territoire des Vosges', + email: 'prms@ddt.vosges.gouv.fr', + telephone: '01 34 22 33 85', + horaires: 'Du lundi au vendredi, de 9 h à 18 h' + ) end end diff --git a/spec/models/dossier_spec.rb b/spec/models/dossier_spec.rb index 312e5d9ab..8a5edc1d3 100644 --- a/spec/models/dossier_spec.rb +++ b/spec/models/dossier_spec.rb @@ -449,7 +449,7 @@ describe Dossier do end.to change(ActionMailer::Base.deliveries, :size).from(0).to(1) mail = ActionMailer::Base.deliveries.last - expect(mail.subject).to eq("Retrouvez votre brouillon pour la démarche \"#{procedure.libelle}\"") + expect(mail.subject).to eq("Retrouvez votre brouillon pour la démarche « #{procedure.libelle} »") expect(mail.html_part.body).to include(dossier_url(dossier)) end