[openstreetmap/openstreetmap-website] Allow more formats for internal latlon search (PR #5147)

Anton Khorev notifications at github.com
Sat Sep 7 00:07:22 UTC 2024


@AntonKhorev commented on this pull request.



> @@ -57,6 +88,7 @@ def test_identify_latlon_ne_d
       "50.06773N, 14.37742E"
     ].each do |code|
       latlon_check code, 50.06773, 14.37742
+      assert_nil @controller.params[:latlon_digits]

This entire file tests `@controller.params` all over the place. It has been doing it for a long time (https://github.com/openstreetmap/openstreetmap-website/commit/aaeca5b5346b01d7ef26a301492804d36049eff8) and you didn't mind adding more recently (https://github.com/openstreetmap/openstreetmap-website/commit/6d217da3b65be4f2973c536fd8fb0765e2a6e450).

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

Message ID: <openstreetmap/openstreetmap-website/pull/5147/review/2287285354 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240906/c4911678/attachment.htm>


More information about the rails-dev mailing list