[openstreetmap/openstreetmap-website] Pluralise user_preferences_controller (#1826)

Andy Allan notifications at github.com
Wed Apr 18 03:28:39 UTC 2018


This is the rails convention for controllers and can make route generation easier.

http://guides.rubyonrails.org/action_controller_overview.html#controller-naming-convention

There's plenty more renaming that can be done, but most need some refactoring around named paths. This one is completely stand-alone!
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Pluralise user_preferences_controller

-- File Changes --

    M .rubocop_todo.yml (2)
    R app/controllers/user_preferences_controller.rb (2)
    M config/routes.rb (10)
    R test/controllers/user_preferences_controller_test.rb (12)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/1826.patch
https://github.com/openstreetmap/openstreetmap-website/pull/1826.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/1826
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180417/2af776ac/attachment.html>


More information about the rails-dev mailing list