good_job 3.12.6 → 3.12.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c529a4f3811213bcf4d534c3fc5c096cac64b944db891eedadd5b553ea77aae3
-  data.tar.gz: fc88a86278de2dd445e463a84f8a1934bc281c976dc585c7db549e5c57ff97ba
+  metadata.gz: 24b3e7de3de7bb0365a230607a0f1f566d431adcd508ce7ee23406164c6528bc
+  data.tar.gz: cd84291baf21cbaabbc543510851481ee835278bd1e844380be99fff749e0702
 SHA512:
-  metadata.gz: d5b076dccfa450de8256870ba502c1f65f8c84fcdd9921772176afe04f1ccc57bd65fd215d46cb6a6c3be8c32570d9e271cbd45b73abdc247c64ef593edab64d
-  data.tar.gz: c243bf54c98d38fe92fa62f104074582968dc757e65f3df611877307b2068d0126d837450745da60a20a80901b52a27af64302694e117c763d268fa3535ed6e5
+  metadata.gz: d79680539f1dfabf4076e313207a54b3f38370f31ca855fdc3cf5bfc20799dd2e6b2495274055a45027459f31439dd11677f270ef823ec6c64b8f44359691312
+  data.tar.gz: 1606e6811d188bd18d56d368fe712871193283c93f3f52c5300052884e4ca1fbb3e8ab502499add0b9364780eca13154ca9cb6c2bfdc8674ca3dd6b157f28209

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [v3.12.7](https://github.com/bensheldon/good_job/tree/v3.12.7) (2023-03-01)
+
+[Full Changelog](https://github.com/bensheldon/good_job/compare/v3.12.6...v3.12.7)
+
+**Fixed bugs:**
+
+- Rename "assets" to "frontends" to avoid Asset Pipeline collisions [\#874](https://github.com/bensheldon/good_job/pull/874) ([bensheldon](https://github.com/bensheldon))
+
+**Closed issues:**
+
+- GoodJob's JavaScript takes precedence over application's JavaScript in GoodJob 3.12.6, Rails 7 [\#873](https://github.com/bensheldon/good_job/issues/873)
+
 ## [v3.12.6](https://github.com/bensheldon/good_job/tree/v3.12.6) (2023-03-01)
 
 [Full Changelog](https://github.com/bensheldon/good_job/compare/v3.12.5...v3.12.6)

data/app/controllers/good_job/frontends_controller.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+module GoodJob
+  class FrontendsController < ActionController::Base # rubocop:disable Rails/ApplicationController
+    skip_after_action :verify_same_origin_request
+
+    STATIC_ASSETS = {
+      css: {
+        bootstrap: GoodJob::Engine.root.join("app", "frontend", "good_job", "vendor", "bootstrap", "bootstrap.min.css"),
+        style: GoodJob::Engine.root.join("app", "frontend", "good_job", "style.css"),
+      },
+      js: {
+        bootstrap: GoodJob::Engine.root.join("app", "frontend", "good_job", "vendor", "bootstrap", "bootstrap.bundle.min.js"),
+        chartjs: GoodJob::Engine.root.join("app", "frontend", "good_job", "vendor", "chartjs", "chart.min.js"),
+        es_module_shims: GoodJob::Engine.root.join("app", "frontend", "good_job", "vendor", "es_module_shims.js"),
+        rails_ujs: GoodJob::Engine.root.join("app", "frontend", "good_job", "vendor", "rails_ujs.js"),
+      },
+    }.freeze
+
+    MODULE_OVERRIDES = {
+      application: GoodJob::Engine.root.join("app", "frontend", "good_job", "application.js"),
+      stimulus: GoodJob::Engine.root.join("app", "frontend", "good_job", "vendor", "stimulus.js"),
+    }.freeze
+
+    def self.js_modules
+      @_js_modules ||= GoodJob::Engine.root.join("app", "frontend", "good_job", "modules").children.select(&:file?).each_with_object({}) do |file, modules|
+        key = File.basename(file.basename.to_s, ".js").to_sym
+        modules[key] = file
+      end.merge(MODULE_OVERRIDES)
+    end
+
+    before_action do
+      expires_in 1.year, public: true
+    end
+
+    def static
+      render file: STATIC_ASSETS.dig(params[:format].to_sym, params[:name].to_sym) || raise(ActionController::RoutingError, 'Not Found')
+    end
+
+    def module
+      raise(ActionController::RoutingError, 'Not Found') if params[:format] != "js"
+
+      render file: self.class.js_modules[params[:name].to_sym] || raise(ActionController::RoutingError, 'Not Found')
+    end
+  end
+end

data/app/views/layouts/good_job/application.html.erb

@@ -8,15 +8,15 @@
   <%= csp_meta_tag %>
 
   <%# Do not use asset tag helpers to avoid paths being overriden by config.asset_host %>
-  <%= tag.link rel: "stylesheet", href: static_asset_path(:bootstrap, format: :css, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
-  <%= tag.link rel: "stylesheet", href: static_asset_path(:style, format: :css, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
+  <%= tag.link rel: "stylesheet", href: frontend_static_path(:bootstrap, format: :css, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
+  <%= tag.link rel: "stylesheet", href: frontend_static_path(:style, format: :css, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
 
-  <%= tag.script "", src: static_asset_path(:bootstrap, format: :js, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
-  <%= tag.script "", src: static_asset_path(:chartjs, format: :js, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
-  <%= tag.script "", src: static_asset_path(:rails_ujs, format: :js, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
+  <%= tag.script "", src: frontend_static_path(:bootstrap, format: :js, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
+  <%= tag.script "", src: frontend_static_path(:chartjs, format: :js, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
+  <%= tag.script "", src: frontend_static_path(:rails_ujs, format: :js, v: GoodJob::VERSION, locale: nil), nonce: content_security_policy_nonce %>
 
-  <%= tag.script "", src: static_asset_path(:es_module_shims, format: :js, v: GoodJob::VERSION, locale: nil), async: true, nonce: content_security_policy_nonce %>
-  <% importmaps = GoodJob::AssetsController.js_modules.keys.index_with { |module_name| module_asset_path(module_name, format: :js, locale: nil, v: GoodJob::VERSION) } %>
+  <%= tag.script "", src: frontend_static_path(:es_module_shims, format: :js, v: GoodJob::VERSION, locale: nil), async: true, nonce: content_security_policy_nonce %>
+  <% importmaps = GoodJob::FrontendsController.js_modules.keys.index_with { |module_name| frontend_module_path(module_name, format: :js, locale: nil, v: GoodJob::VERSION) } %>
   <%= tag.script({ imports: importmaps }.to_json.html_safe, type: "importmap", nonce: content_security_policy_nonce) %>
   <%= tag.script "", type: "module", nonce: content_security_policy_nonce do %> import "application"; <% end %>
 </head>

data/config/routes.rb

@@ -27,8 +27,8 @@ GoodJob::Engine.routes.draw do
 
   resources :processes, only: %i[index]
 
-  scope :assets, controller: :assets do
-    get "modules/:name", action: :module, as: :module_asset, constraints: { format: 'js' }
-    get "static/:name", action: :static, as: :static_asset, constraints: { format: %w[css js] }
+  scope :frontend, controller: :frontends do
+    get "modules/:name", action: :module, as: :frontend_module, constraints: { format: 'js' }
+    get "static/:name", action: :static, as: :frontend_static, constraints: { format: %w[css js] }
   end
 end

data/lib/good_job/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 module GoodJob
   # GoodJob gem version.
-  VERSION = '3.12.6'
+  VERSION = '3.12.7'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: good_job
 version: !ruby/object:Gem::Version
-  version: 3.12.6
+  version: 3.12.7
 platform: ruby
 authors:
 - Ben Sheldon
@@ -318,30 +318,30 @@ files:
 - CHANGELOG.md
 - LICENSE.txt
 - README.md
-- app/assets/good_job/application.js
-- app/assets/good_job/modules/charts.js
-- app/assets/good_job/modules/checkbox_toggle.js
-- app/assets/good_job/modules/document_ready.js
-- app/assets/good_job/modules/live_poll.js
-- app/assets/good_job/modules/popovers.js
-- app/assets/good_job/modules/toasts.js
-- app/assets/good_job/style.css
-- app/assets/good_job/vendor/bootstrap/bootstrap.bundle.min.js
-- app/assets/good_job/vendor/bootstrap/bootstrap.min.css
-- app/assets/good_job/vendor/chartjs/chart.min.js
-- app/assets/good_job/vendor/es_module_shims.js
-- app/assets/good_job/vendor/rails_ujs.js
-- app/assets/good_job/vendor/stimulus.js
 - app/charts/good_job/scheduled_by_queue_chart.rb
 - app/controllers/good_job/application_controller.rb
-- app/controllers/good_job/assets_controller.rb
 - app/controllers/good_job/batches_controller.rb
 - app/controllers/good_job/cron_entries_controller.rb
+- app/controllers/good_job/frontends_controller.rb
 - app/controllers/good_job/jobs_controller.rb
 - app/controllers/good_job/processes_controller.rb
 - app/filters/good_job/base_filter.rb
 - app/filters/good_job/batches_filter.rb
 - app/filters/good_job/jobs_filter.rb
+- app/frontend/good_job/application.js
+- app/frontend/good_job/modules/charts.js
+- app/frontend/good_job/modules/checkbox_toggle.js
+- app/frontend/good_job/modules/document_ready.js
+- app/frontend/good_job/modules/live_poll.js
+- app/frontend/good_job/modules/popovers.js
+- app/frontend/good_job/modules/toasts.js
+- app/frontend/good_job/style.css
+- app/frontend/good_job/vendor/bootstrap/bootstrap.bundle.min.js
+- app/frontend/good_job/vendor/bootstrap/bootstrap.min.css
+- app/frontend/good_job/vendor/chartjs/chart.min.js
+- app/frontend/good_job/vendor/es_module_shims.js
+- app/frontend/good_job/vendor/rails_ujs.js
+- app/frontend/good_job/vendor/stimulus.js
 - app/helpers/good_job/application_helper.rb
 - app/models/concerns/good_job/filterable.rb
 - app/models/concerns/good_job/reportable.rb

data/app/controllers/good_job/assets_controller.rb

@@ -1,45 +0,0 @@
-# frozen_string_literal: true
-module GoodJob
-  class AssetsController < ActionController::Base # rubocop:disable Rails/ApplicationController
-    skip_after_action :verify_same_origin_request
-
-    STATIC_ASSETS = {
-      css: {
-        bootstrap: GoodJob::Engine.root.join("app", "assets", "good_job", "vendor", "bootstrap", "bootstrap.min.css"),
-        style: GoodJob::Engine.root.join("app", "assets", "good_job", "style.css"),
-      },
-      js: {
-        bootstrap: GoodJob::Engine.root.join("app", "assets", "good_job", "vendor", "bootstrap", "bootstrap.bundle.min.js"),
-        chartjs: GoodJob::Engine.root.join("app", "assets", "good_job", "vendor", "chartjs", "chart.min.js"),
-        es_module_shims: GoodJob::Engine.root.join("app", "assets", "good_job", "vendor", "es_module_shims.js"),
-        rails_ujs: GoodJob::Engine.root.join("app", "assets", "good_job", "vendor", "rails_ujs.js"),
-      },
-    }.freeze
-
-    MODULE_OVERRIDES = {
-      application: GoodJob::Engine.root.join("app", "assets", "good_job", "application.js"),
-      stimulus: GoodJob::Engine.root.join("app", "assets", "good_job", "vendor", "stimulus.js"),
-    }.freeze
-
-    def self.js_modules
-      @_js_modules ||= GoodJob::Engine.root.join("app", "assets", "good_job", "modules").children.select(&:file?).each_with_object({}) do |file, modules|
-        key = File.basename(file.basename.to_s, ".js").to_sym
-        modules[key] = file
-      end.merge(MODULE_OVERRIDES)
-    end
-
-    before_action do
-      expires_in 1.year, public: true
-    end
-
-    def static
-      render file: STATIC_ASSETS.dig(params[:format].to_sym, params[:name].to_sym) || raise(ActionController::RoutingError, 'Not Found')
-    end
-
-    def module
-      raise(ActionController::RoutingError, 'Not Found') if params[:format] != "js"
-
-      render file: self.class.js_modules[params[:name].to_sym] || raise(ActionController::RoutingError, 'Not Found')
-    end
-  end
-end