ETQ Admin je peux configurer une démarche avec accusé de lecture #10190

lisa-durand · 2024-03-25T13:13:43Z

closes #9786

L'admin a une card et une page pour configurer l'accusé de lecture

L'usager recoit un mail sans la décision qui l'invite à se connecter à DS

L'usager une fois sur DS doit accepter l'accusé de lecture

Dans cette PR, on a la possibilté pour l'admin d'activer un accusé de lecture. Il faut donc masquer la décison dans plusieurs parties de l'app, tant que l'accusé n'a pas été accepté par l'usager.

les badges "accepté" "refusé" et "classé sans suite" deviennent "traité"
dans la liste des dossiers de l'usager, les dossiers sont masqués du filtre par état pour les dossiers terminés
le message automatique de la messagerie est plus générique
les mails de notifs pour les usagers et pour les tiers sont plus génériques

emsnytech · 2024-03-26T19:42:13Z

L'idéal c'est que l'accusé de lecture ne soit parametrable que pour les décisions.
Cela ne concerne donc que:
Les commentaires mis en annexe de la décision
Les trois mails de décision
Il est probable que les agents puissent souhaiter aussi le paramétrer pour des mails normaux ou bien quand on est à sva lors de demandes de pièces complémentaires.
Est ce que ce cas on été vu ?

lisa-durand · 2024-03-27T09:26:03Z

L'idéal c'est que l'accusé de lecture ne soit parametrable que pour les décisions. Cela ne concerne donc que: Les commentaires mis en annexe de la décision Les trois mails de décision Il est probable que les agents puissent souhaiter aussi le paramétrer pour des mails normaux ou bien quand on est à sva lors de demandes de pièces complémentaires. Est ce que ce cas on été vu ?

Je ne suis pas sur de comprendre, pour le contexte cette demande vient de notre dernier forum ouvert et l'idée de l'accusé de lecture est bien de masquer la décision. Du coup je ne comprends pas les autres cas de figure ou en tout cas ils ne seront pas pris en compte.

Rappel du contexte:

toutes les démarches avec voies de recours (si refus, l'usager a x jours pour faire un recours) => Il faut pouvoir prouver que l'usager a eu l'info

ici tous les services de certifcations, agréments (DRIEETS) ~ 30K dossiers idf

emsnytech · 2024-03-27T10:19:22Z

On est Ok sur le cas général, testons avec le cas d'usage sur les 3 mails de décision. On verra si les demandes se font plus large ensuite.
De mon expérience... j'en vois d'autres...

app/views/administrateurs/procedures/accuse_lecture.html.haml

LeSim

Première lecture, ca a l'air bien cool.
Prochaine étape je la fais tourner en locale.

LeSim · 2024-04-09T13:02:15Z

app/views/notification_mailer/send_accuse_lecture_notification.html.haml

+  = t("layouts.mailers.accuse_lecture.second_part")
+%p
+  = t("layouts.mailers.accuse_lecture.third_part")
+  = link_to 'Démarches Simplifiées.', dossier_url(@dossier)


Suggested change

= link_to 'Démarches Simplifiées.', dossier_url(@dossier)

= link_to Current.application_name, dossier_url(@dossier)

LeSim · 2024-04-09T13:11:00Z

app/views/notification_mailer/send_accuse_lecture_notification.html.haml

+  = t(:best_regards, scope: [:views, :shared, :greetings])
+  %br
+    = t('layouts.mailers.signature.team')
+    #{APPLICATION_NAME.gsub(".","&#8288;.").html_safe}


Current.application_name à la place d'application name.

Pourrais tu retirer ce gsub.html_safe étrange ? (De manière générale le html_safe me faire peur. )

LeSim · 2024-04-10T09:39:59Z

app/helpers/dossier_helper.rb

+      tag.span 'traité', role: 'status', class: class_names(
+        'fr-badge fr-badge--sm fr-badge--no-icon' => true,
+        alignment_class => true
+      )


tu peux pas faire directe

Suggested change

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'

?

E-L-T

C'est top 👍 , j'ai juste fait des mini retours de forme par ci par là

E-L-T · 2024-04-10T09:07:54Z

app/views/administrateurs/procedures/accuse_lecture.html.haml

+      = render Dsfr::AlertComponent.new(state: :info, size: :sm, extra_class_names: 'fr-mb-2w') 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 Démarches Simplifiées pour en prendre connaissance et en accuser lecture.


petites typos : email au lieu de mail, plus mettre des vraies apostrophes

E-L-T · 2024-04-10T09:25:50Z

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

+        - if dossier.procedure.accuse_lecture? && !dossier.accuse_lecture_agreement_at.present?
+          = render Dsfr::CalloutComponent.new(title: nil) do |c|
+            - c.with_body do
+              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.


remplacer procédure par démarche. Et petit doute sur l'expression "vous acceptez l'accusé de lecture", plus les apostrophes

E-L-T · 2024-04-10T09:28:01Z

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


+            - 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


utiliser external attributes ?

E-L-T · 2024-04-10T10:22:55Z

spec/controllers/instructeurs/dossiers_controller_spec.rb

+
+        it 'creates a commentaire' do
+          expect { subject }.to change { Commentaire.count }.by(1)
+          expect(dossier_accuse_lecture.commentaires.last.body).to eq("<p>Bonjour,</p><p>Nous vous informons qu'une décision sur votre dossier a été rendue.</p>Cordialement,<br>#{procedure_accuse_lecture.service.nom}")


ptet inverser l'ordre dans la phrase ?

LeSim

Super pr ! En ptits commits et tout et tout.

Un ptit point fonctionnel : J'ai l'impression que tu n'affiches pas la date de l'accusé de lecture dans l'interface instructeur, ne faudrait-il pas le rajouter ?

LeSim · 2024-04-11T07:44:52Z

app/views/administrateurs/procedures/accuse_lecture.html.haml

+      = render Dsfr::AlertComponent.new(state: :info, size: :sm, extra_class_names: 'fr-mb-2w') 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.


proposition pour expliquer l'utilité de cette fonctionnalité

Suggested change

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 %br

Cette fonctionnalité permet à l'instructeur de connaître la date de lecture de la décision finale par l'usager %br %br

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.

et si t'es motivée, il faudrait transformer les ' par des ’

LeSim · 2024-04-11T07:47:52Z

app/components/procedure/card/accuse_lecture_component/accuse_lecture_component.html.haml

+      - if @procedure.accuse_lecture.present?
+        %p.fr-badge.fr-badge--success Activé
+      - else
+        %p.fr-badge.fr-badge--info À configurer


peut être

Suggested change

%p.fr-badge.fr-badge--info À configurer

%p.fr-badge.fr-badge--info Désactivé

serait plus adapté (mm texte que l'attestation).

LeSim · 2024-04-11T07:51:37Z

app/views/notification_mailer/send_accuse_lecture_notification.html.haml

+
+%p
+  = t("layouts.mailers.accuse_lecture.first_part",
+  dossier_id: number_with_delimiter(@dossier.id))


number_with_delimiter ❤️

lisa-durand · 2024-04-15T08:45:09Z

Super pr ! En ptits commits et tout et tout.

Un ptit point fonctionnel : J'ai l'impression que tu n'affiches pas la date de l'accusé de lecture dans l'interface instructeur, ne faudrait-il pas le rajouter ?

Ah oui bien vu - j'ai rajouté un callout pour les instructeurs

Co-authored-by: LeSim <mail@simon.lehericey.net>

lisa-durand force-pushed the add-AR-feature-ldu branch from e6fff26 to e319be8 Compare March 26, 2024 14:58

lisa-durand force-pushed the add-AR-feature-ldu branch from e319be8 to 4088bce Compare March 27, 2024 10:06

lisa-durand force-pushed the add-AR-feature-ldu branch 8 times, most recently from f4ac02d to 6885d5d Compare April 3, 2024 09:21

lisa-durand marked this pull request as ready for review April 3, 2024 09:24

LeSim reviewed Apr 9, 2024

View reviewed changes

app/views/administrateurs/procedures/accuse_lecture.html.haml Outdated Show resolved Hide resolved

LeSim reviewed Apr 10, 2024

View reviewed changes

E-L-T approved these changes Apr 10, 2024

View reviewed changes

LeSim approved these changes Apr 11, 2024

View reviewed changes

add AR component and page on admin view

ee91f47

lisa-durand force-pushed the add-AR-feature-ldu branch from 4c51a90 to 1bd2e76 Compare April 15, 2024 08:48

lisa-durand added 9 commits April 15, 2024 11:15

create AR email and send it at the right time

ae08044

add AR in user view - hide decision on first dossier tab

8eda069

add AR in user view - hide decision on second dossier tab

e486f78

hide decision displayed in badge for user

8fe9b17

change wording for expired_at to remove decision in it

b2e48b4

hide decision displayed in messagerie with auto commentaire

5bbc4ca

hide dossier with accuse de lecture in user list when using state filter

1cf3db5

fix specs

18602e0

hide decision from email for tiers if procedure has accuse de lecture

71268c7

lisa-durand and others added 2 commits April 15, 2024 11:15

Correction Sim PR - use dynamic application name

13aeb4f

Co-authored-by: LeSim <mail@simon.lehericey.net>

small adjustments from PR feedbacks

a29719a

lisa-durand force-pushed the add-AR-feature-ldu branch from 1bd2e76 to 0ec80f3 Compare April 15, 2024 09:15

add instructeur feedback on accuse de lecture agreement

c6e2e64

lisa-durand force-pushed the add-AR-feature-ldu branch from 0ec80f3 to c6e2e64 Compare April 15, 2024 10:10

lisa-durand added this pull request to the merge queue Apr 16, 2024

Merged via the queue into main with commit 7a80574 Apr 16, 2024
17 checks passed

lisa-durand deleted the add-AR-feature-ldu branch April 16, 2024 10:07

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ETQ Admin je peux configurer une démarche avec accusé de lecture #10190

ETQ Admin je peux configurer une démarche avec accusé de lecture #10190

lisa-durand commented Mar 25, 2024 •

edited

Loading

emsnytech commented Mar 26, 2024

lisa-durand commented Mar 27, 2024

emsnytech commented Mar 27, 2024

LeSim left a comment

LeSim Apr 9, 2024

LeSim Apr 9, 2024

LeSim Apr 10, 2024

E-L-T left a comment

E-L-T Apr 10, 2024

E-L-T Apr 10, 2024

E-L-T Apr 10, 2024

E-L-T Apr 10, 2024

LeSim left a comment

LeSim Apr 11, 2024

LeSim Apr 11, 2024

LeSim Apr 11, 2024

lisa-durand commented Apr 15, 2024

	= link_to 'Démarches Simplifiées.', dossier_url(@dossier)
	= link_to Current.application_name, dossier_url(@dossier)

	%p.fr-badge.fr-badge--info À configurer
	%p.fr-badge.fr-badge--info Désactivé

ETQ Admin je peux configurer une démarche avec accusé de lecture #10190

ETQ Admin je peux configurer une démarche avec accusé de lecture #10190

Conversation

lisa-durand commented Mar 25, 2024 • edited Loading

L'admin a une card et une page pour configurer l'accusé de lecture

L'usager recoit un mail sans la décision qui l'invite à se connecter à DS

L'usager une fois sur DS doit accepter l'accusé de lecture

emsnytech commented Mar 26, 2024

lisa-durand commented Mar 27, 2024

emsnytech commented Mar 27, 2024

LeSim left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

E-L-T left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LeSim left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

lisa-durand commented Apr 15, 2024

lisa-durand commented Mar 25, 2024 •

edited

Loading