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

Show full error messages #34

Closed
joeytalbot opened this issue Jun 7, 2022 · 3 comments
Closed

Show full error messages #34

joeytalbot opened this issue Jun 7, 2022 · 3 comments

Comments

@joeytalbot
Copy link
Contributor

When a route fails, it would be very helpful to show the error message that explains why the route failed. E.g.

"Too short: journeys must be longer than 4 metres. (Your requested journey was 0 metres)."

This is especially needed for the batch routing, where a single failed route prevents the entire process from completing.

@Robinlovelace
Copy link
Collaborator

May be fixed by #35

@mvl22
Copy link
Member

mvl22 commented Jun 7, 2022

Use the 'errors' field in the /v2/batchroutes.jobdata output.

We recommend you output those to the terminal so the person running the batch job gets that summary of the errors.

@Robinlovelace
Copy link
Collaborator

Makes sense, that would definitely be an improvement. For some reason I thought the error message would be in the json field as valid json but that's a valid place for it to be.

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

3 participants