[openstreetmap/openstreetmap-website] Add demo links for routing engine directions Fixes #6387 (PR #6693)

Areeb Siddiqui notifications at github.com
Mon Jan 12 18:32:44 UTC 2026


@SykoDaedalus commented on this pull request.



> @@ -80,11 +80,20 @@
             }
           })
         });
+        const demoQuery = new URLSearchParams({ profile: DEMO_PARAMS[modeId] });

> Then I'd rather add this config to the `OSM.Directions.addEngine` lines than having a separate dict in different places.

Valhalla and osrm will then need 3 params in addEngine function while graphhopper we can work with vehicleType (the existing 2 params). Should I change it to 3 params for all of them (to maintain consistency) or only in 2 where we need it?

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

Message ID: <openstreetmap/openstreetmap-website/pull/6693/review/3652144518 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260112/598e9407/attachment.htm>


More information about the rails-dev mailing list