[openstreetmap/openstreetmap-website] Already logged in warning (PR #5957)

Anton Khorev notifications at github.com
Sun Apr 27 03:38:51 UTC 2025


@AntonKhorev commented on this pull request.



>      $("#referer").val($("#referer").val() + location.hash);
+    $("#referer_link").prop("href", $("#referer_link").prop("href") + location.hash);

`$("#referer_link")[0].hash = location.hash;` requires the link to exist and we're not checking that.

Changed to `$("#referer_link").prop("hash", location.hash);`.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5957/review/2796908032 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250426/c6b19e80/attachment-0001.htm>


More information about the rails-dev mailing list