Skip to content

Commit

Permalink
Better clarify need to override internal_methods
Browse files Browse the repository at this point in the history
Co-authored-by: Eebs Kobeissi <ebrahim.kobeissi@gmail.com>
  • Loading branch information
carlosantoniodasilva and eebs committed Oct 10, 2023
1 parent 9a08620 commit c146b25
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion app/controllers/devise_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,9 @@ def _prefixes #:nodoc:
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
Expand Down

0 comments on commit c146b25

Please sign in to comment.