[openstreetmap/openstreetmap-website] Append one table per route leg (PR #5720)

Marwin Hochfelsner notifications at github.com
Sat Feb 22 14:19:32 UTC 2025


### Description
I modified the directions engine interface to return the steps grouped into legs and create a turn instruction table for each leg. This change isn't visible to the user, as there isn't a third endpoint yet.
### How has this been tested?
So much browser-based that I initially forgot to update the testing stub
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Merge sidebar content append calls
  * Append one table per route leg

-- File Changes --

    M app/assets/javascripts/index/directions.js (64)
    M app/assets/javascripts/index/directions/fossgis_osrm.js (14)
    M app/assets/javascripts/index/directions/fossgis_valhalla.js (8)
    M app/assets/javascripts/index/directions/graphhopper.js (25)
    M test/system/directions_test.rb (4)

-- Patch Links --

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

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

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


More information about the rails-dev mailing list