[openstreetmap/openstreetmap-website] Load js controller modules only when needed (PR #7147)

Marwin Hochfelsner notifications at github.com
Tue Jun 23 17:56:41 UTC 2026


@hlfan commented on this pull request.



> @@ -56,6 +56,10 @@
 
     return page;
   };
+};
+
+{

I personally don't need 200 more lines in this diff, but I have it removed in the follow-up clean-up branch [`cleaned-modules`](https://github.com/hlfan/openstreetmap-website/compare/modularise..cleaned-modules). I can append that commit if you want.

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

Message ID: <openstreetmap/openstreetmap-website/pull/7147/review/4555820011 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260623/a83941b4/attachment.htm>


More information about the rails-dev mailing list