Merge pull request #2450 from bitzesty/press-summaries-specs-fix

Move `after_action` to the concern [production]
bitzesty · Aug 1, 2023 · a911d9b · a911d9b
2 parents 936b288 + 9769e03
commit a911d9b
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 2 deletions.
diff --git a/app/controllers/admin/press_summaries_controller.rb b/app/controllers/admin/press_summaries_controller.rb
@@ -1,4 +1,3 @@
 class Admin::PressSummariesController < Admin::BaseController
-  after_action :log_event, only: [:create, :update, :submit, :signoff]
   include PressSummaryMixin
 end
diff --git a/app/controllers/assessor/press_summaries_controller.rb b/app/controllers/assessor/press_summaries_controller.rb
@@ -1,4 +1,3 @@
 class Assessor::PressSummariesController < Assessor::BaseController
-  after_action :log_event, only: [:create, :update, :submit, :signoff]
   include PressSummaryMixin
 end
diff --git a/app/controllers/concerns/press_summary_mixin.rb b/app/controllers/concerns/press_summary_mixin.rb
@@ -4,6 +4,7 @@ module PressSummaryMixin
   def self.included(base)
     base.before_action :load_form_answer
     base.before_action :load_press_summary, except: :create
+    base.after_action :log_event, only: [:create, :update, :submit, :signoff]
   end
 
   def create