Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Affiche la section des pièces jointes en haut pour les instructrices et viseuses #1598

Open
wants to merge 2 commits into
base: staging
Choose a base branch
from

Conversation

alemangui
Copy link
Collaborator

@alemangui alemangui commented Feb 4, 2025

Closes #1559

Contexte

L'affichage des pièces jointes n'était pas adapté pour les instructrices et viseuses, qui ont besoin d'une interface plus compacte en haut de l'onglet produit.

UI

Vu que la page DeclarationSummary est partagée entre pro, instructrice et viseuse, j'ai ajouté un paramètre useCompactAttachmentView. Lors qu'il est à true, la section des pièces jointes utilise un DsfrFileDownload et remonte en haut.

Certains fichiers en base de données ont des noms très longs, j'ai donc ajouté une fonction dans utils/string.js qui permet de remplacer le milieu d'une chaîne de caractères par afin de s'assurer que ladite chaîne ne dépasse pas une certaine longueur. La raison pour laquelle on coupe le milieu et non pas la fin est que normalement l'extension du fichier est une information utile placée à la fin du nom de fichier.

L'endroit du DsfrFileDownload où d'habitude on trouve l'extension du fichier affiche le type de pièce jointe - une information plus utile afin de repérer les étiquettes du reste. De plus, l'extension du fichier est dans le nom.

Backend

Le DsfrFileDownload affiche également le la taille du fichier, que j'ai du ajouter dans le serializer des pièces jointes et dans le modèle en tant que propriété.

J'en ai profité pour ajouter (au modèle seulement) une propriété pour le format/extension dans le cas ou d'autres endroits de l'application utilisent le DsfrFileDownload et veulent afficher l'extension directement de l'API. Cette propriété n'est pas exposé pour l'instant dans le serializer.

Démo

image

@alemangui alemangui requested review from pletelli and hfroot February 4, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

En tant qu'instructrice et viseur je souhaite voir les PJ en haut de l'onglet produit
1 participant