[openstreetmap/openstreetmap-website] Add yearly template and CalendarLabel plugin for month labels (PR #5906)

Tom Hughes notifications at github.com
Thu Apr 10 15:54:06 UTC 2025


@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/2757363679 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250410/e8f7e120/attachment.htm>


More information about the rails-dev mailing list