etalab / transport-site

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

Suppression de l'affichage "indisponibilité" pour le JDD de Lyon Métropole #4052

Closed Brewennn closed 1 month ago

Brewennn commented 1 month ago

Re salut @AntoineAugusti et @cyrilmorin,

En lien avec l'issue #4051 relative aux données de Lyon Métropole, je vous propose de ne pas afficher ni sur la page du JDD ni dans l'API que le jeu de données est indisponible, ou à l'inverse on affiche que c'est disponible par défaut. Etant donné qu'il y aura le bandeau pour indiquer que l'information sur ces jeux n'est pas disponible, on laisse les réutilisateurs se connecter sur l'open data de Lyon pour qu'ils se fassent une idée.

Si c'est ok, je me demande comment l'indiquer dans la partie "resources" dans l'API pour le champ 'is_availabe :' est-ce qu'on le met en 'True' par défaut ou change-t-on la nature de ce champ en format text pour indiquer un attribut "Not_communicated" en plus de "False" et "True" ? Je pense que la première solution serait plus simple.

cyrilmorin commented 1 month ago

Hello @Brewenn,

Je suis un peu embêté philosophiquement pour afficher une ressource comme "Disponible" alors qu'on ne le sait pas. Notre bandeau est un plus mais a objectivement moins d'impact et d'utilité que nos api/stats/score/monitoring/notifications ... qui seraient tous impactés. En plus il se peut qu'on ait d'autres publications soumises à authentification qu'on afficherait Indisponible j'imagine (?)

@AntoineAugusti de manière générale, suite à notre tentative d'accès au fichier/service soumis à authentification, est-ce qu'on a un retour différencié du système requêté genre Erreur 403 Forbidden ? Si oui ça pourrait être bien d'avoir un statut intermédiaire genre "Unknown" effectivement entre True et False.

@Brewennn Je préfère ta version 2 :) mais je laisse @AntoineAugusti nous préciser la complexité !

Dispo pour en parler

AntoineAugusti commented 1 month ago

En plus il se peut qu'on ait d'autres publications soumises à authentification qu'on afficherait Indisponible j'imagine (?)

Je serais pour pouvoir contrôler ce comportement à l'aide d'un tag pour ne pas faire de la métropole de Lyon un cas spécial. Ceci peut être fait par le biais de https://github.com/etalab/transport-site/issues/4051 qui permet d'indiquer qu'une authentification est nécessaire pour accéder aux données.

Je suis un peu embêté philosophiquement pour afficher une ressource comme "Disponible" alors qu'on ne le sait pas

Je suis plus réservé, comme @cyrilmorin, sur le fait de donner un passe-droit sur la disponibilité des données quand une authentification est en place. Sans s'être authentifié et avoir tenté de télécharger on ne connaît pas l'état de disponibilité des données. Accorder un ✅ me parait un peu précipité.

Ne pas avoir téléchargé la donnée a plus d'impacts que l'état de disponibilité, quand on regarde une page de JDD, on constate que :

Je me demande donc quel degré d'adaptation nous souhaitons faire.

Brewennn commented 1 month ago

@AntoineAugusti et @cyrilmorin,

L'objectif est bien de faire le strict minimum pour Lyon et juste enlever la notion "indisponible" qui tend à induire les utilisateurs en erreur car même avec le bandeau, on pourrait croire que le jeu n'est pas disponible ni sur le PAN ni sur l'open data de Lyon Métropole. Au regard de ce que vous avez dit, j'y vois deux solutions :

J'opte pour la seconde solution plus simple.

cyrilmorin commented 1 month ago

Pour un quick win 2nde option également de mon côté !

Brewennn commented 1 month ago

Dans ce cas on peut clôturer cet issue.

AntoineAugusti commented 1 month ago

4051 est adapté pour pouvoir traiter ce ticket