Skip to content

Commit

Permalink
[www] Fix valhalla routing instructions
Browse files Browse the repository at this point in the history
Clicking on a route step and non-transit routing was not rendering.

I broke this while working on the travelmux API unification. I
unwittingly committed some code to the wrong file that made valhalla api
responses camelcase.

OTP is camelCase and Valhalla is snake_case. I decided to adopt
camelCase for the unified responses, but my intention was always to
leave the original _valhalla and _otp responses untouched so that legacy
clients could easily migrate.
  • Loading branch information
michaelkirk committed Mar 26, 2024
1 parent b44b3e4 commit 5a8ab89
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion services/travelmux/src/valhalla/valhalla_api.rs
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ pub struct Leg {
}

#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
#[serde(rename_all(serialize = "camelCase", deserialize = "snake_case"))]
pub struct Maneuver {
pub instruction: String,
pub cost: f64,
Expand Down

0 comments on commit 5a8ab89

Please sign in to comment.