From c146b25f31f81386c8ab71e0cb8ea2fc3723cad5 Mon Sep 17 00:00:00 2001 From: Carlos Antonio da Silva Date: Tue, 10 Oct 2023 17:24:07 -0300 Subject: [PATCH] Better clarify need to override `internal_methods` Co-authored-by: Eebs Kobeissi --- app/controllers/devise_controller.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/controllers/devise_controller.rb b/app/controllers/devise_controller.rb index 0bcd17a7..1c3b4162 100644 --- a/app/controllers/devise_controller.rb +++ b/app/controllers/devise_controller.rb @@ -33,7 +33,9 @@ class DeviseController < Devise.parent_controller.constantize end end - # Override internal methods to exclude `_prefixes` since we override it above. + # Override internal methods to exclude `_prefixes` from action methods since + # we override it above. + # # There was an intentional change in Rails 7.1 that will allow it to become # an action method because it's a public method of a non-abstract controller, # but we also can't make this abstract because it can affect potential actions