[openstreetmap/openstreetmap-website] Set getRoute default function parameters (PR #6446)
Marwin Hochfelsner
notifications at github.com
Thu Mar 26 18:50:03 UTC 2026
hlfan left a comment (openstreetmap/openstreetmap-website#6446)
> OK it looks like it is now just a technical refactoring to simplify the code, and on that basis it looks fine to me.
Yeah this was originally mostly a technical refactoring with some route retriggering mechanisms added, but doing it the right way is more involved. That's why I removed the second commit, as it isn't straightforward for users at all. See e.g. [this comment](https://github.com/openstreetmap/openstreetmap-website/pull/6446#issuecomment-3405757362):
> I think this is a slight improvement, but doesn't solve the problem.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6446?email_source=notifications&email_token=AAK2OLO3WJ2GSWJRVJV5C6T4SV3VXA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJTG42DANJUHE4KM4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOS2XA4S7MNXW23LFNZ2F633QMVXF6Y3MNFRWW#issuecomment-4137405498
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6446/c4137405498 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260326/4fe22e54/attachment.htm>
More information about the rails-dev
mailing list