@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/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
