@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/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to