[openstreetmap/openstreetmap-website] Don't apply extra encodeURIComponent to lat/lon search params (PR #5595)
Anton Khorev
notifications at github.com
Mon Feb 3 10:16:46 UTC 2025
See https://github.com/openstreetmap/openstreetmap-website/pull/5592#discussion_r1938530806
In practice this should make no difference because lat/lon don't contain any characters that need escaping. But there's still no reason for attempting to escape them twice.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/5595
-- Commit Summary --
* Don't apply extra encodeURIComponent to lat/lon search params
-- File Changes --
M app/assets/javascripts/index/contextmenu.js (2)
M app/assets/javascripts/index/search.js (2)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/5595.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5595.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5595
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/5595 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250203/a3f5b411/attachment-0001.htm>
More information about the rails-dev
mailing list