[openstreetmap/openstreetmap-website] Move diary comments index action to comments controller (PR #4889)

Anton Khorev notifications at github.com
Mon Jun 10 13:16:27 UTC 2024


There was a diary comment model but no controller. Everything related to comments was in the diary entries controller.

Here I'm moving `diary_entries#comments` to `diary_comments#index`. There are at least three other comment actions, I'll move them later.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Move diary comments index action to comments controller

-- File Changes --

    M app/abilities/ability.rb (3)
    A app/controllers/diary_comments_controller.rb (27)
    M app/controllers/diary_entries_controller.rb (15)
    R app/views/diary_comments/index.html.erb (4)
    M config/locales/en.yml (15)
    M config/routes.rb (2)
    A test/controllers/diary_comments_controller_test.rb (63)
    M test/controllers/diary_entries_controller_test.rb (52)

-- Patch Links --

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

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4889
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4889 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240610/4b3eba1b/attachment-0001.htm>


More information about the rails-dev mailing list