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

Tom Hughes notifications at github.com
Wed Jun 10 17:45:37 UTC 2026


tomhughes left a comment (openstreetmap/openstreetmap-website#7147)

> Does `app/assets/javascripts/index_modules/history-changesets-layer.js` need to get the `export` line too?

No because it's not exported as a module itself, it's included into `history.js`.

Of course it winds up polluting the global namespace (well the `OSM` namespace) with functions that are only used in the new `index_history` module.

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

Message ID: <openstreetmap/openstreetmap-website/pull/7147/c4672833580 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260610/34e5a14b/attachment.htm>


More information about the rails-dev mailing list