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

Documentation on ManeuverType is outdated #4666

Open
erikvullings opened this issue Mar 28, 2024 · 0 comments
Open

Documentation on ManeuverType is outdated #4666

erikvullings opened this issue Mar 28, 2024 · 0 comments
Labels

Comments

@erikvullings
Copy link

While using Jelmer's Rust-based client to query Valhalla's routing interface, I received many JSON serialization errors due to the fact that not all ManeuverType options were implemented. As it turned out, however, even the Valhalla API documentation does not mention them all. The actual list of values, as implemented in the code, can be found here. Therefore, please add the missing enum values to your documentation, specifically:

  • kElevatorEnter (value 39)
  • kStepsEnter
  • kEscalatorEnter
  • kBuildingEnter
  • kBuildingExit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants