ETQ Usager utilisant un lecteur d'écran, je veux que le titre du lien de modification d'un dossier déposé reprenne l'intitulé visible

[inseo]

Ajout du numéro de dossier au lien de téléchargement de l'attestation

Après

Avant

Ajout de la traduction au lien de téléchargement du dossier

Après

Avant

Ajout d'une étiquette à la navigation d'un dossier

Après

Avant

Harmonisation du titre d'un lien et de son intitulé visible

Après

Avant

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.07%. Comparing base (87e0def) to head (e7e58e4).
Report is 153 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main   #11623       +/-   ##
===========================================
+ Coverage   57.29%   83.07%   +25.78%     
===========================================
  Files        1249     1249               
  Lines       27278    27278               
  Branches     5591     5591               
===========================================
+ Hits        15629    22662     +7033     
+ Misses      11649     4616     -7033     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[colinux]

Pour que le test sur le mailer passe, j'ai l'impression qu'il faut aussi passer dossier_id dans app/models/mails/initiated_mail.rb ligne 32 (vu depuis app/mailers/notification_mailer.rb ligne 115)

[inseo]

Pour que le test sur le mailer passe, j'ai l'impression qu'il faut aussi passer dossier_id dans app/models/mails/initiated_mail.rb ligne 32 (vu depuis app/mailers/notification_mailer.rb ligne 115)

Je pense que ça a réglé un truc : l'erreur n'est plus la même. :D

[colinux]

Le test échoue ici car il ya une situation (au moins) où on affiche le lien sans être dans le contexte du dossier: quand on affiche la page d'aperçu du message de fin de dépôt aux administrateurs. Dans ce cas on veut leur afficher le lien visible (sans être actif pour autant) pour qu'ils voient à quoi ça ressemble. Et donc ton changement échoue car le nom du fichier à télécharger ne peut pas être génér2.

Pour éviter d'avoir trop d'opérateurs ternaires sur plusieurs lignes j'ai l'impression qu'on pourrait faire ça :

- if dossier.present?
  = link_to "#{t('views.users.dossiers.merci.download_dossier')} (PDF)",  dossier_path(dossier, format: :pdf), download: "#{t('views.users.dossiers.show.header.download', dossier_id: dossier.id)}", class: 'fr-btn fr-btn--secondary fr-btn--icon-left fr-icon-download-line', **external_link_attributes
- else # contexte d'aperçu du message en fin dépôt pour les admins
  = link_to "#{t('views.users.dossiers.merci.download_dossier')} (PDF)",  "#", download: "Mon dossier", class: 'fr-btn fr-btn--secondary fr-btn--icon-left fr-icon-download-line', **external_link_attributes

[inseo]

Ton correctif a bien corrigé l'erreur.
Mais une nouvelle l'a (encore) remplacée. :s

[inseo]

Je me permets un petit up. :)

[colinux]

finalement les tests sont passés, je merge