[openstreetmap/openstreetmap-website] Add valhalla routing (PR #3847)

Christian notifications at github.com
Fri Dec 16 11:43:14 UTC 2022


Fixes #3843 


I reverted https://github.com/openstreetmap/openstreetmap-website/commit/9340c461732ef7387dd453ea3fcf5e64ea782ab4, removed references to Mapzen, and added two new maneuvers to the instructions map (`kMergeLeft`, `kMergeRight`). I also renamed the fossgis engine to `OSRM` so the distinction is clearer.


You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/3847

-- Commit Summary --

  * Revert "Remove dependencies on mapzen services"
  * reverted 9340c46
  * reverted prettier autoformatting
  * corrected commit revert
  * corrected commit revert
  * got valhalla working again
  * renamed mapzen.js to valhalla.js
  * renamed fossgis engine to osrm
  * renamed fossgis.js to osrm.js
  * added new merge left and right maneuvers to valhalla engine
  * removed legacy reference to mapquest engine, renamed intruction map variable

-- File Changes --

    R app/assets/javascripts/index/directions/osrm.js (12)
    A app/assets/javascripts/index/directions/valhalla.js (112)
    M app/assets/javascripts/osm.js.erb (1)
    M app/controllers/application_controller.rb (2)
    M config/locales/en.yml (3)
    M config/settings.yml (1)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/3847.patch
https://github.com/openstreetmap/openstreetmap-website/pull/3847.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3847
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/3847 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20221216/da0bd02b/attachment.htm>


More information about the rails-dev mailing list