@tomhughes commented on this pull request.
This is adding a whole load of names to the global namespace and I'm not sure
any of it is necessary...
Is there are reason all the things declared at global scope before the event
listener can't be declared inside it as they were before?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5906#pullrequestreview-2757363679
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5906/review/2757363...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
