From a29719a15cd1f486b7123182d9006663ea8c7d5d Mon Sep 17 00:00:00 2001 From: Lisa Durand Date: Thu, 11 Apr 2024 13:00:57 +0200 Subject: [PATCH] small adjustments from PR feedbacks --- .../accuse_lecture_component.fr.yml | 2 +- .../accuse_lecture_component.html.haml | 2 +- app/helpers/dossier_helper.rb | 5 +---- .../procedures/accuse_lecture.html.haml | 12 +++++++++--- .../send_accuse_lecture_notification.html.haml | 4 ++-- .../users/dossiers/show/_status_overview.html.haml | 2 +- spec/views/users/dossiers/demande.html.haml_spec.rb | 4 ++-- .../dossiers/show/_status_overview.html.haml_spec.rb | 2 +- 8 files changed, 18 insertions(+), 15 deletions(-) diff --git a/app/components/dossiers/accuse_lecture_component/accuse_lecture_component.fr.yml b/app/components/dossiers/accuse_lecture_component/accuse_lecture_component.fr.yml index 509404dbe..dd8a4b038 100644 --- a/app/components/dossiers/accuse_lecture_component/accuse_lecture_component.fr.yml +++ b/app/components/dossiers/accuse_lecture_component/accuse_lecture_component.fr.yml @@ -1,4 +1,4 @@ --- fr: - text_accuse_lecture: Cette procédure est soumise à un accusé de lecture. En demandant l'affichage de la décision prise sur votre dossier, vous acceptez l'accusé de lecture et par là même le démarrage du délai légal en cas de recours. + text_accuse_lecture: Cette démarche est soumise à un accusé de lecture. En demandant l’affichage de la décision prise sur votre dossier, vous acceptez l’accusé de lecture et par là même le démarrage du délai légal en cas de recours. btn_accuse_lecture: Je souhaite afficher la décision diff --git a/app/components/procedure/card/accuse_lecture_component/accuse_lecture_component.html.haml b/app/components/procedure/card/accuse_lecture_component/accuse_lecture_component.html.haml index 48729b558..8a722dc72 100644 --- a/app/components/procedure/card/accuse_lecture_component/accuse_lecture_component.html.haml +++ b/app/components/procedure/card/accuse_lecture_component/accuse_lecture_component.html.haml @@ -4,7 +4,7 @@ - if @procedure.accuse_lecture.present? %p.fr-badge.fr-badge--success Activé - else - %p.fr-badge.fr-badge--info À configurer + %p.fr-badge.fr-badge--info Désactivé %div %h3.fr-h6.fr-mt-10v= t('.title') %p.fr-tile-subtitle= t('.subtitle') diff --git a/app/helpers/dossier_helper.rb b/app/helpers/dossier_helper.rb index 8d83959e7..7394a71e6 100644 --- a/app/helpers/dossier_helper.rb +++ b/app/helpers/dossier_helper.rb @@ -80,10 +80,7 @@ module DossierHelper def status_badge_user(dossier, alignment_class = '') if dossier.hide_info_with_accuse_lecture? - tag.span 'traité', role: 'status', class: class_names( - 'fr-badge fr-badge--sm fr-badge--no-icon' => true, - alignment_class => true - ) + tag.span 'traité', role: 'status', class: "fr-badge fr-badge--sm fr-badge--no-icon #{alignment_class}" else status_badge(dossier.state, alignment_class) end diff --git a/app/views/administrateurs/procedures/accuse_lecture.html.haml b/app/views/administrateurs/procedures/accuse_lecture.html.haml index 6e87c306c..13e998dec 100644 --- a/app/views/administrateurs/procedures/accuse_lecture.html.haml +++ b/app/views/administrateurs/procedures/accuse_lecture.html.haml @@ -9,10 +9,16 @@ %h1.page-title Accusé de lecture - = render Dsfr::AlertComponent.new(state: :info, size: :sm, extra_class_names: 'fr-mb-2w') do |c| + = render Dsfr::CalloutComponent.new(title: nil) do |c| - c.with_body do %p - Pour les démarches avec voies de recours, il est possible d'activer l'accusé de lecture. L'usager n'a plus accès à la décision finale par mail, mais il doit se connecter sur la plateforme #{Current.application_name} pour en prendre connaissance et en accuser lecture. + Pour les démarches avec voies de recours, il est possible d’activer l’accusé de lecture. + %br + %p + Cette fonctionnalité permet à l’instructeur de connaître la date de lecture de la décision finale par l’usager. + %br + %p + L’usager n’a plus accès à la décision finale par mail, mais il doit se connecter sur la plateforme #{Current.application_name} pour en prendre connaissance et en accuser lecture. %ul.fr-toggle__list %li @@ -24,7 +30,7 @@ = render Dsfr::ToggleComponent.new(form: f, target: :accuse_lecture, title: "Accusé de lecture de la démarche", - hint: "L’accusé de lecture est à activer uniquement pour les démarches avec voies de recours car il complexifie l'accès à la décision finale pour les usagers", + hint: "L’accusé de lecture est à activer uniquement pour les démarches avec voies de recours car il complexifie l’accès à la décision finale pour les usagers", opt: {"checked" => @procedure.accuse_lecture}) .padded-fixed-footer diff --git a/app/views/notification_mailer/send_accuse_lecture_notification.html.haml b/app/views/notification_mailer/send_accuse_lecture_notification.html.haml index 58f1f4ba1..3e3a81573 100644 --- a/app/views/notification_mailer/send_accuse_lecture_notification.html.haml +++ b/app/views/notification_mailer/send_accuse_lecture_notification.html.haml @@ -14,10 +14,10 @@ = t("layouts.mailers.accuse_lecture.second_part") %p = t("layouts.mailers.accuse_lecture.third_part") - = link_to 'Démarches Simplifiées.', dossier_url(@dossier) + = link_to APPLICATION_NAME, dossier_url(@dossier) %p = t(:best_regards, scope: [:views, :shared, :greetings]) %br = t('layouts.mailers.signature.team') - #{APPLICATION_NAME.gsub(".","⁠.").html_safe} + = APPLICATION_NAME diff --git a/app/views/users/dossiers/show/_status_overview.html.haml b/app/views/users/dossiers/show/_status_overview.html.haml index 61e727227..b1c4f6ea6 100644 --- a/app/views/users/dossiers/show/_status_overview.html.haml +++ b/app/views/users/dossiers/show/_status_overview.html.haml @@ -72,7 +72,7 @@ - if dossier.attestation.present? .action - = link_to attestation_dossier_path(dossier), class: "fr-btn fr-icon-download-line fr-btn--icon-left", target: '_blank', rel: 'noopener' do + = link_to attestation_dossier_path(dossier), class: "fr-btn fr-icon-download-line fr-btn--icon-left", **external_link_attributes do = t('views.users.dossiers.show.status_overview.accepte_attestation') diff --git a/spec/views/users/dossiers/demande.html.haml_spec.rb b/spec/views/users/dossiers/demande.html.haml_spec.rb index b45a788b1..333db5574 100644 --- a/spec/views/users/dossiers/demande.html.haml_spec.rb +++ b/spec/views/users/dossiers/demande.html.haml_spec.rb @@ -64,7 +64,7 @@ describe 'users/dossiers/demande', type: :view do let(:dossier) { create(:dossier, :accepte, :with_motivation) } it 'displays the motivation' do - expect(rendered).not_to have_text('Cette procédure est soumise à un accusé de lecture.') + expect(rendered).not_to have_text('Cette démarche est soumise à un accusé de lecture.') expect(rendered).to have_text('Motivation') end end @@ -73,7 +73,7 @@ describe 'users/dossiers/demande', type: :view do let(:dossier) { create(:dossier, :accepte, :with_motivation, procedure: create(:procedure, :accuse_lecture)) } it 'display information about accuse de lecture and not the motivation' do - expect(rendered).to have_text('Cette procédure est soumise à un accusé de lecture.') + expect(rendered).to have_text('Cette démarche est soumise à un accusé de lecture.') expect(rendered).not_to have_text('Motivation') end end diff --git a/spec/views/users/dossiers/show/_status_overview.html.haml_spec.rb b/spec/views/users/dossiers/show/_status_overview.html.haml_spec.rb index 501a8f946..9f1bfcfab 100644 --- a/spec/views/users/dossiers/show/_status_overview.html.haml_spec.rb +++ b/spec/views/users/dossiers/show/_status_overview.html.haml_spec.rb @@ -98,6 +98,6 @@ describe 'users/dossiers/show/_status_overview', type: :view do it { is_expected.not_to have_selector('.status-explanation .sans-suite') } it { is_expected.not_to have_text(dossier.motivation) } - it { is_expected.to have_text('Cette procédure est soumise à un accusé de lecture.') } + it { is_expected.to have_text('Cette démarche est soumise à un accusé de lecture.') } end end