[openstreetmap/openstreetmap-website] Set iD map param with array (PR #5632)

Tom Hughes notifications at github.com
Wed Feb 12 18:23:48 UTC 2025


@tomhughes approved this pull request.

Thanks for addressing all the review comment - this looks good to me now.

The only thing that slightly confuses me is the need for parentheses around the destructuring on lines 20 and 22 as it seems to work without that when I try it in the node console but eslint does fail to parse it unless you include them!



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

Message ID: <openstreetmap/openstreetmap-website/pull/5632/review/2612845509 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250212/7bf69e5d/attachment.htm>


More information about the rails-dev mailing list