search

etalab / transport-site

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

Correctif pour étiquette "Non disponible" incorrecte sur le flux SIRI de Boulogne #4284

Closed thbar closed 1 week ago

thbar commented 3 weeks ago

Voir:

Le serveur SIRI est techniquement bien joignable (si on met de côté, comme beaucoup d'autres flux, l'absence de requestor_ref https://github.com/etalab/transport-site/issues/4254), cette PR modifie l'availability check pour qu'il indique aux utilisateurs que c'est bien le cas (malgré le code 500 pas très pratique).

Test rapide sous iex

> Transport.AvailabilityChecker.available?("SIRI", "https://saes.ratpdev.com/ctb") 
true

(contre false avant la PR).

/cc @Brewennn pour information

Écran "avant"

AntoineAugusti commented 3 weeks ago

Merci pour le fix 🙏

Tu confirmes que tu vas laisser ainsi les données historiques d'indisponibilité et les scores ? Ça peut être compliqué/long à nettoyer.

thbar commented 2 weeks ago

Tu confirmes que tu vas laisser ainsi les données historiques d'indisponibilité et les scores ? Ça peut être compliqué/long à nettoyer.

@AntoineAugusti oui tout à fait, je ne vais pas chercher à corriger ça. Merci pour le reste !

AntoineAugusti commented 2 weeks ago

La traduction est là https://github.com/etalab/transport-site/blob/8b4f026cac10cbdbb27ae180a7966c97ed0ca5af/apps/transport/priv/gettext/fr/LC_MESSAGES/page-dataset-details.po#L343

J'étais sur mobile avant, désolé de pas avoir partagé ce lien.

thbar commented 1 week ago

Peux-tu mettre à jour la traduction expliquant le fonctionnement de ce mécanisme sur la page resource#details ? On distingue le cas classique SIRI

J'ai fait ça dans ff302fcd42d82a07ee63e4eb5b2eb3cc2555500c (merci pour le lien PO).