-
Notifications
You must be signed in to change notification settings - Fork 30
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
Informations de complétude d'un GTFS #3779
Conversation
0d8f451
to
62f14bb
Compare
f1511c6
to
6b4b0c7
Compare
Merge queue setting changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci pour la PR, c’est tout bon à part deux trois suggestions de locales en français.
Dans quelle mesure on peut avoir ça aussi dans le validateur à la demande ? C’est pas les mêmes vues, non ? Ou si, on créée une pseudo resource lors de la validation à la demande ?
apps/transport/test/transport/validators/gtfs_transport_validator_test.exs
Show resolved
Hide resolved
apps/transport/priv/gettext/fr/LC_MESSAGES/page-dataset-details.po
Outdated
Show resolved
Hide resolved
apps/transport/priv/gettext/fr/LC_MESSAGES/page-dataset-details.po
Outdated
Show resolved
Hide resolved
@@ -320,16 +322,18 @@ defmodule Transport.Validators.GTFSTransport do | |||
@doc """ | |||
Outputs a tag if at least 80% of GTFS routes have a custom color. | |||
|
|||
iex> has_route_colors_tag(%{"lines_with_custom_color_count" => 8, "lines_count" => 10}) | |||
iex> has_route_colors_tag(%{"stats" => %{"routes_with_custom_color_count" => 8, "routes_count" => 10}}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je veux bien l’avis de @thbar sur quelque chose non directement lié à cette PR : les tags sont en français en raison de la recherche, ça a été introduit par @fchabouis ici https://github.com/etalab/transport-site/pull/2683/files#diff-ba66fa740d0d2e1c3f2a155321eba90321801019d60e0720aa102b660476ea9fR296 C’est dans ton radar pour la refonte de la recherche ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci pour le lien - oui on finira sûrement par intégrer ça aussi, et en français je pense :-)
Co-authored-by: Vincent Degove <vincent.degove@beta.gouv.fr>
transport-site/apps/transport/lib/transport_web/templates/validation/show.html.heex Line 19 in 1a1ec68
|
En lien avec #3739.
Utilise les statistiques ajoutées dans etalab/transport-validator#192
Conformément aux specs, affiche des informations détaillées des éléments compris dans un GTFS sur la page
resource#details
pour les GTFS.