Remove MailersController overrides

All this code relates to ActionMailer previews and it is what we think causes the previews to fail in Rails > 7.0.8. Prepending any objectm, as was done here has a high price and is not needed for the previews to work in Rails. v2 will use a different approach to this.
dxw · Apr 19, 2024 · f3ac3d6 · f3ac3d6
1 parent a59716e
commit f3ac3d6
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 93 deletions.
diff --git a/lib/mail/notify.rb b/lib/mail/notify.rb
@@ -8,7 +8,6 @@
 require "mail/notify/personalisation"
 require "mail/notify/mailer"
 require "mail/notify/message"
-require "mail/notify/mailers_controller"
 
 Mail::Message.include Mail::Notify::Message
 

diff --git a/lib/mail/notify/engine.rb b/lib/mail/notify/engine.rb
@@ -8,12 +8,6 @@ class Engine < Rails::Engine
       initializer "mail-notify.add_delivery_method", before: "action_mailer.set_configs" do
         ActionMailer::Base.add_delivery_method(:notify, Mail::Notify::DeliveryMethod)
       end
-
-      initializer "mail-notify.action_controller" do
-        ActiveSupport.on_load(:action_controller, run_once: true) do
-          Rails::MailersController.prepend(Mail::Notify::MailersController)
-        end
-      end
     end
   end
 end
diff --git a/lib/mail/notify/mailers_controller.rb b/lib/mail/notify/mailers_controller.rb
diff --git a/spec/controllers/mailers_controller_spec.rb b/spec/controllers/mailers_controller_spec.rb