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

endpoint not working - gtfs_ride_stops/list?gtfs_ride_ids=<id> #26

Closed
NoamGaash opened this issue May 22, 2023 · 9 comments
Closed

endpoint not working - gtfs_ride_stops/list?gtfs_ride_ids=<id> #26

NoamGaash opened this issue May 22, 2023 · 9 comments
Assignees

Comments

@NoamGaash
Copy link
Member

the endpoint -
https://open-bus-stride-api.hasadna.org.il/gtfs_ride_stops/list?gtfs_ride_ids=42934600
keeps the client side hanging without a response.
removing the gtfs_ride_ids field works just fine

@OriHoch
Copy link
Contributor

OriHoch commented May 23, 2023

@NoamGaash This results in a heavy query on the DB, I suggest to always add gtfs date ranges to your your queries, then it will return very fast:

https://open-bus-stride-api.hasadna.org.il/gtfs_ride_stops/list?gtfs_ride_ids=42934600&gtfs_stop__date_from=2023-05-20&gtfs_stop__date_to=2023-05-30&gtfs_route__date_from=2023-05-20&gtfs_route__date_to=2023-05-30

@ShayAdler
Copy link
Contributor

@NoamGaash just as thinking of ideas for small donations, do you think it would be helpful to add a limit in the API for that (fail immediately when not enough filters are supplied?)

@NoamGaash
Copy link
Member Author

That's a great idea. It should be implemented to return error 400

@ohaval
Copy link
Contributor

ohaval commented May 26, 2024

Hey, I want to take and work on this little issue.

@NoamGaash
Copy link
Member Author

@ohaval great, thanks!

@OriHoch
Copy link
Contributor

OriHoch commented May 27, 2024

@ohaval you can contact me on hasadna slack if you want guidance / help with this task

@OriHoch
Copy link
Contributor

OriHoch commented Jun 19, 2024

fixed in #42

@OriHoch OriHoch closed this as completed Jun 19, 2024
@OriHoch
Copy link
Contributor

OriHoch commented Jun 19, 2024

@NoamGaash @ohaval fixed to allow requests for a specific gtfs ride id

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

No branches or pull requests

4 participants