[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