search

mapzen / documentation

Configuration files, tools, and content to build Mapzen's documentation
https://mapzen.com/documentation
MIT License
25 stars 22 forks source link

Some mobility doc links in index.md are broken #211

Closed rmglennon closed 6 years ago

rmglennon commented 7 years ago

Some links in index.md are broken, and it doesn't really make sense. I'm sure these worked before because I would have tested the homepage.

[Mapzen Turn-by-Turn](turn-by-turn-overview.md) tries to link to https://mapzen.com/documentation/mobility/turn-by-turn-overview.md when it should go to https://mapzen.com/documentation/mobility/turn-by-turn/overview/

[Optimized Route service](optimized_route/api-reference.md) tries to link to https://mapzen.com/documentation/mobility/optimized_route/api-reference.md when it should go to https://mapzen.com/documentation/mobility/optimized/api-reference/

works (matrix/api-reference)

This is odd because index.md is in the same structure as turn-by-turn-overview, and links should use the .md extension. Also, this shows that the mz-redirects work to send everything to a mobility URL, but not the mz-renames to change the optimized_route folder name.

Spotted by @randymeech.

migurski commented 7 years ago

Hm, these are all ones that we rename in the Mobility config.

migurski commented 7 years ago

So we have options.

  1. Continue on the rename-train and make our code smarter about following names around.
  2. Take a hint and change the paths upstream.
rmglennon commented 7 years ago

See also https://github.com/valhalla/valhalla-docs/issues/127

to reorganize the content.