@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/2612845...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
