etalab / transport-site

Rendre disponible, valoriser et améliorer les données transports
https://transport.data.gouv.fr
194 stars 30 forks source link

Affichage cohérent des cartes GeoJSON #2301

Closed AntoineAugusti closed 5 months ago

AntoineAugusti commented 2 years ago

Suite à https://github.com/etalab/transport-site/pull/2217 de @fchabouis les GeoJSONs disposent de visualisations sur la page de détails de chaque ressource.

Pourtant, certains jeux de données ont une visualisation du GeoJSON proposé en plus de la liste des ressources.

Voir

Envisager de revoir DatasetView.get_resource_to_display en conséquence

https://github.com/etalab/transport-site/blob/3e44c53fa7ed354b66869ffe915378d560d2c356/apps/transport/lib/transport_web/templates/dataset/details.html.eex#L43-L62

AntoineAugusti commented 2 years ago

https://github.com/etalab/transport-site/blob/24c1481de30936451156838a0c2335bade4d9332/apps/transport/client/javascripts/resource-viz.js#L351-L354 a son rôle aussi

ptitfred commented 5 months ago

Dans le cas de Strasbourg, c'est lié au pattern utilisé pour les URLs dans leur API :

> {:ok, zfe_strasbourg} = DB.Dataset.get_by_slug("zone-a-faibles-emissions-mobilite")
> TransportWeb.DatasetView.get_resource_to_display(zfe_strasbourg).url
"https://data.strasbourg.eu/api/explore/v2.1/catalog/datasets/zfe_emprise/exports/geojson"
ptitfred commented 5 months ago

Peut-être devrait-on plutôt s'appuyer sur le nom de la resource ?

ptitfred commented 5 months ago

Je vais tenir compte du format, comme fait pour d'autres types de visualisations.