Amelioration: ETQ instructeur, afficher les délais d'instruction estimés aux usagers + Tech: migration HAML → ERB stats#13335
Merged
Conversation
mfo
changed the title
mfo
force-pushed
the
rework-stats
branch
14 times, most recently
from
June 23, 2026 11:56
417abca to
bfe2331
Compare
3 tasks
Allow super-admins to hide processing delay estimations from users, similar to the existing estimated_duration_visible toggle. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Display the instruction delay accordion even on EN TEST procedures, using placeholder text instead of computed values. Also fix typos in the i18n strings (add "de", "Pour les", comma after "échanges"). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add a mention next to the "Temps de traitement usuel" title on the instructeur stats page to inform them this data is shown to users. The mention is not displayed on the usager stats page. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
mfo
marked this pull request as ready for review
June 23, 2026 13:42
colinux
approved these changes
Jun 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Probleme
Les Administrateurs & instructeurs n'ont pas de visibilité sur le fait que les délais d'instruction sont montrés aux usagers. De plus, sur les procédures EN TEST (brouillon), l'accordion des délais et le suivi de dossier n'affichent rien car il n'y a pas encore de données de traitement — l'admin ne peut pas prévisualiser le rendu.
Enfin, il manque un levier super-admin pour désactiver l'affichage des délais de traitement estimés sur une procédure donnée.
Closes #13330
Solution
4 changements fonctionnels :
Super-admin : ajout d'une colonne
estimated_processing_duration_visible(boolean, default true) surprocedures, exposée dans le dashboard Administrate. Permet de masquer les délais estimés pour les usagers uniquement.Accordion EN TEST : sur les procédures brouillon, l'accordion "Quels sont les délais d'instruction" s'affiche avec des placeholders
[indication du délai]/[indication du délai moyen]pour que l'admin puisse prévisualiser le rendu.Dossier suivi EN TEST : même placeholders sur la page de suivi d'un dossier en_construction d'une procédure brouillon.
Stats instructeur : ajout de la mention "(indiqué aux usagers)" à côté du titre "Temps de traitement usuel" sur la page stats instructeur uniquement. L'instructeur voit toujours les données de traitement, même quand le flag est désactivé.
Correction au passage de typos i18n dans les traductions du composant (
est :→est de).2 migrations HAML → ERB :
instructeurs/procedures/stats.html.haml→ ERB + extraction i18n du titreshared/procedures/_stats.html.haml→ ERBScreenshots
ETQ admin, sur une de mes démarches en brouillon, prévisualiser les délais sur une procédure EN TEST (accordion)
ETQ usager, voir les délais estimés sur le suivi d'un dossier EN TEST
ETQ instructeur, voir que les délais sont indiqués aux usagers (flag activé)
ETQ super-admin, activer/désactiver la visibilité des délais estimés
Migration HAML → ERB
Skill
/haml-migrationinstructeurs/procedures/stats — ✅ identique au byte
Validation : formatter herb ✅, tests ✅, apostrophes ✅
Extraction i18n du titre "Statistiques de la démarche" →
t('instructeurs.dossiers.header.banner.statistics')(clé existante).Avant :
Après :
Couverture visuelle (1/1 utilisation) :
localhost:3000/procedures/234/stats— page stats instructeurshared/procedures/_stats — ✅ identique au byte
Validation : formatter herb ✅, tests ✅, apostrophes ✅
Pas de texte en dur (toutes les clés i18n existaient déjà).
Avant :
Après :
Couverture visuelle (1/2 utilisations) :
localhost:3000/procedures/234/stats— page stats instructeurlocalhost:3000/statistiques/:path— page stats usager (même partial, pas de données différentes)Voir tous les screenshots de comparaison
Generated with Claude Code