[openstreetmap/openstreetmap-website] Use login_path instead of explicit controller and actions (#3125)

Andy Allan notifications at github.com
Wed Mar 10 14:41:39 UTC 2021


I'm working on some refactoring of users_controller, and this is a minor part of that work that can be split off and reviewed/merged separately.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/3125

-- Commit Summary --

  * Use login_path instead of explicit controller and actions

-- File Changes --

    M .rubocop_todo.yml (2)
    M app/controllers/application_controller.rb (4)
    M app/controllers/messages_controller.rb (4)
    M app/controllers/users_controller.rb (8)
    M app/views/browse/changeset.html.erb (2)
    M app/views/diary_entries/show.html.erb (2)
    M test/controllers/changesets_controller_test.rb (4)
    M test/controllers/diary_entries_controller_test.rb (8)
    M test/controllers/site_controller_test.rb (4)
    M test/controllers/traces_controller_test.rb (6)
    M test/controllers/users_controller_test.rb (14)
    M test/integration/client_applications_test.rb (2)
    M test/integration/user_login_test.rb (52)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/3125.patch
https://github.com/openstreetmap/openstreetmap-website/pull/3125.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3125
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20210310/c65f5a82/attachment.htm>


More information about the rails-dev mailing list