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

Areeb Siddiqui notifications at github.com
Fri Jan 16 17:33:37 UTC 2026


SykoDaedalus left a comment (openstreetmap/openstreetmap-website#6693)

> Ok, the changes overall look good to me, but the commit message is a bit vague.
> 
> I think it would be clearer with
> 
> * all the feature changes in one commit (maybe titled alike the PR)
> * and the added newlines for readability in another.
> 
> Final nit while you're rebasing: the JS spread syntax is usually written without a trailing space, aka `"..._processDirections"` instead of `"... _processDirections"`.

separated features and newlines into 2 different commits. Fixed the trailing space issue

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

Message ID: <openstreetmap/openstreetmap-website/pull/6693/c3761091942 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260116/1fe4d021/attachment.htm>


More information about the rails-dev mailing list