@tomhughes commented on this pull request.
> @@ -33,9 +33,15 @@ OSM = {
LAYER_DEFINITIONS: <%= MapLayers::full_definitions("config/layers.yml",
:legends => "config/legend.yml").to_json %>,
- RTL_TEXT_PLUGIN: <%=
javascript_path("@mapbox/mapbox-gl-rtl-text/dist/mapbox-gl-rtl-text.js").to_json
%>,
-
- MAKE_PLURAL_CARDINALS: <%=
javascript_path("make-plural/cardinals.js").to_json %>,
+ MODULE_PATHS: <%=
+ %i[changeset directions element export history home index new_note note
query search]
+ .each_with_object({}) { |module_name, hash|
hash[:"index_#{module_name}"] = "index_modules/#{module_name}" }
+ .merge(
+ :mapbox_rtl_text =>
"@mapbox/mapbox-gl-rtl-text/dist/mapbox-gl-rtl-text.js",
+ :make_plural_cardinals => "make-plural/cardinals.js"
+ )
+ .transform_values { |path| javascript_path(path).sub('javascripts/',
'assets/') }
Well I think they should be... It seems to work for me in the console:
```ruby
open-street-map(dev):004> helper.javascript_path("matomo")
=>
"/assets/matomo-5f12566701d19841982934ab1470493a5322b844bd3e9cf0f0b5fadccac275ba.js"
```
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7147#discussion_r3448827178
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/7147/review/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
