-
Notifications
You must be signed in to change notification settings - Fork 6
Feat: gbfs validator, validation report UI styling + gbfs java validator types #1422
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
Conversation
|
*Lighthouse ran on https://mobility-feeds-dev--pr-1422-l8gkn31v.web.app/ * (Desktop)
*Lighthouse ran on https://mobility-feeds-dev--pr-1422-l8gkn31v.web.app/feeds * (Desktop)
*Lighthouse ran on https://mobility-feeds-dev--pr-1422-l8gkn31v.web.app/feeds/gtfs/mdb-2126 * (Desktop)
*Lighthouse ran on https://mobility-feeds-dev--pr-1422-l8gkn31v.web.app/feeds/gtfs_rt/mdb-2585 * (Desktop)
*Lighthouse ran on https://mobility-feeds-dev--pr-1422-l8gkn31v.web.app/gbfs/gbfs-flamingo_porirua * (Desktop)
|
|
Preview Firebase Hosting URL: https://mobility-feeds-dev--pr-1422-l8gkn31v.web.app |
|
@Alessandro100 This makes sense to me! I'm wondering if there are any licensing considerations we should factor in with OSM. cc @davidgamez and @cka-y here - @Alessandro100 would we be making any osm backend data publicly available with the map or is it literally just the map visual? |
|
Quick question: In GbfsValidator.yaml, version: "0.1" is used, shall we bump it up? |
There is a SPIKE ticket to further investigate the map implementation. From the early preview we'll be calling the gbfs data directly from the source. It doesn't look like we'll have any OSM problems here |
This file comes directly from the gbfs-validator-java repo (copy and paste) I wouldn't want to modify it |
qcdyx
left a comment
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.
LGTM!
davidgamez
left a comment
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.
LGTM!


Summary:
closes https://github.com/MobilityData/product-tasks/issues/163
Added the type from the gbfs-validator openapi file + styled the validation report page. Updated the dark mode theme contrast text to be more accessible with the contrast
Expected behavior:
When you go to the validation report page, that all the elements look cohesive, look good on light mode, dark mode, and all screen resolutions
Testing tips:
Go to the validation report page
gbfs-validator?AutoDiscoveryUrl=https%3A%2F%2Fbackend.citiz.fr%2Fpublic%2Fprovider%2F19%2Fgbfs%2Fv3.0%2Fgbfs.jsonand play around with the UI (screen sizes, light / dark mode). Report things that could be improved or are not clearNote
All values are hard coded and going to be changed
Please make sure these boxes are checked before submitting your pull request - thanks!
./scripts/api-tests.shto make sure you didn't break anything