search

datagouv / data.gouv.fr

Ce dépôt rassemble les tickets techniques qui portent sur data.gouv.fr.
https://www.data.gouv.fr
76 stars 14 forks source link

Masquer/griser le bouton prévisualiser pour les CSV trop lourds #615

Closed jonathanfallon closed 1 year ago

jonathanfallon commented 2 years ago

Le problème

La prévisualisation des fichiers trop lourds retourne une erreur dans la modale image

Pour éviter ce comportement, il serait possible de masquer ou griser (disabled) le bouton de prévisualisation (visible sur l'image ci-dessous)

image

maudetes commented 2 years ago

Merci du signalement. En effet, il me semble préférable de désactiver le bouton de prévisualisation si la ressource est trop lourde. Est-ce qu'il serait possible d'avoir le lien vers le jeu de données en question?

ThomasG77 commented 2 years ago

Lien vers la ressource qui fait plantée https://www.data.gouv.fr/fr/datasets/les-donnees-contextuelles-des-intercommunalites-et-autres-structures-territoriales/#resources

L'idéal ne serait pas forcément de griser/masquer mais "streamer" ou de faire un "limit" et limiter la prévisualisation aux n premières lignes. Pas bien identifié où cela serait possible entre les briques https://github.com/etalab/csvapi, https://github.com/opendatateam/udata-tabular-preview/, https://github.com/etalab/csvapi. L'approche peut aussi varier si on est capable de passer la taille du fichier à prévisualiser aux autres briques (mais après risque de couplage plus fort)

maudetes commented 2 years ago

A voir ce qu'on veut faire en effet, ça me semble pertinent de n'afficher plutôt que les premières lignes. Mais il y a bien une restriction appliquée sur la taille aujourd'hui, qui doit coincer sur cette ressource. L'affichage de la prévisualisation est dépendante de ce code, qui prend en compte content-length ou la taille déclarée.

agarrone commented 1 year ago

Je peux clore @nicolaskempf57 non ?

nicolaskempf57 commented 1 year ago

Normalement on avait déjà une vérification sur la taille sur l'ancienne préviz. Je vérifie si jamais il n'y a pas un décalage entre la taille max de CSVAPI et celle acceptée pour une previz et je te dis.

nicolaskempf57 commented 1 year ago

Je confirme qu'on a toujours le problème sur dev avec le meme jdd : https://dev.data.gouv.fr/en/datasets/les-donnees-contextuelles-des-intercommunalites-et-autres-structures-territoriales/

nicolaskempf57 commented 1 year ago

PR en cours sur le dépôt d'infra pour corriger le problème.

nicolaskempf57 commented 1 year ago

C'est finalisé coté infra