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

Areeb Siddiqui notifications at github.com
Mon Jan 12 16:45:31 UTC 2026


@SykoDaedalus commented on this pull request.



> @@ -174,15 +175,23 @@
           // invalidate cache
           cachedHints = [];
         }
-
+        const demoQuery = new URLSearchParams({ srv: DEMO_PARAMS[modeId] });

> Could this be done with Array.indexOf?

 I could use Array.indexOf(), but I think keeping the mapping makes it clearer what's happening and keeps the style consistent with other engines. It might help in future to understand the vehicle-to-parameter relationship at a glance?

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

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


More information about the rails-dev mailing list