Skip to content
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

Fix vehicle_types.json wrongly required #85 #86

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

tdelmas
Copy link
Collaborator

@tdelmas tdelmas commented Oct 5, 2022

Fix #85

  • A station_id field on vehicles in free_bike_status.json will not mark vehicle_types.json anymore as required, as the spec doesn't require it.

Code cleaning:

  • freeBikeStatusHasVehicleId var is renamed freeBikeStatusHasVehicleTypeId to better reflect what it check.

https://gbfs.nextbike.net/maps/gbfs/v2/nextbike_le/gbfs.json is not marked invalid anymore.

@netlify
Copy link

netlify bot commented Oct 5, 2022

Deploy Preview for competent-payne-690ca9 failed.

Name Link
🔨 Latest commit 274582f
🔍 Latest deploy log https://app.netlify.com/sites/competent-payne-690ca9/deploys/633d78320484090009bac58f

@netlify
Copy link

netlify bot commented Oct 5, 2022

Deploy Preview for gbfs-validator ready!

Name Link
🔨 Latest commit 274582f
🔍 Latest deploy log https://app.netlify.com/sites/gbfs-validator/deploys/633d7832d004790008acf68d
😎 Deploy Preview https://deploy-preview-86--gbfs-validator.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@PierrickP PierrickP requested a review from isabelle-dr October 5, 2022 12:31
@PierrickP
Copy link
Collaborator

Hello @nbdh can you test it ?

@netlify
Copy link

netlify bot commented Oct 5, 2022

Deploy Preview for wizardly-engelbart-5c48ca failed.

Name Link
🔨 Latest commit 274582f
🔍 Latest deploy log https://app.netlify.com/sites/wizardly-engelbart-5c48ca/deploys/633d7832ac34b600096903ae

@netlify
Copy link

netlify bot commented Oct 5, 2022

Deploy Preview for kind-pike-a3f3f8 failed.

Name Link
🔨 Latest commit 274582f
🔍 Latest deploy log https://app.netlify.com/sites/kind-pike-a3f3f8/deploys/633d783214c65900085f8aa3

@nbdh
Copy link
Contributor

nbdh commented Oct 5, 2022

Hi @PierrickP,

I've checked https://deploy-preview-86--gbfs-validator.netlify.app/?url=https%3A%2F%2Fgbfs.nextbike.net%2Fmaps%2Fgbfs%2Fv2%2Fnextbike_le%2Fgbfs.json

and can confirm that vehicle_types.json is not marked as required anymore and consequently the feed is considered valid.

Thanks for the fix @tdelmas and the good work everyone!

@netlify
Copy link

netlify bot commented Oct 5, 2022

Deploy Preview for wizardly-lichterman-770f54 canceled.

Name Link
🔨 Latest commit 274582f
🔍 Latest deploy log https://app.netlify.com/sites/wizardly-lichterman-770f54/deploys/633d78323e927300076cd2e2

@netlify
Copy link

netlify bot commented Oct 5, 2022

Deploy Preview for unruffled-hugle-914373 ready!

Name Link
🔨 Latest commit 274582f
🔍 Latest deploy log https://app.netlify.com/sites/unruffled-hugle-914373/deploys/633d78323d1bab000979aff7
😎 Deploy Preview https://deploy-preview-86--unruffled-hugle-914373.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@josee-sabourin josee-sabourin merged commit 55e0d00 into MobilityData:master Oct 5, 2022
@tdelmas tdelmas deleted the fix_85 branch October 5, 2022 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vehicle_types.json wrongly required
4 participants