[openstreetmap/openstreetmap-website] Use resourceful route for message reply (PR #5458)

Anton Khorev notifications at github.com
Thu Jan 2 08:48:36 UTC 2025


Creates a nested `reply` resource for messages.

Changes reply path from `/messages/:id/reply` to `/messages/:id/reply/new` because it's a new reply.

You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Use resourceful route for message reply

-- File Changes --

    M app/abilities/ability.rb (2)
    A app/controllers/messages/replies_controller.rb (50)
    M app/controllers/messages_controller.rb (37)
    M app/views/messages/show.html.erb (2)
    M config/locales/en.yml (5)
    M config/routes.rb (3)
    A test/controllers/messages/replies_controller_test.rb (69)
    M test/controllers/messages_controller_test.rb (65)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/5458 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250102/7d2329dc/attachment.htm>


More information about the rails-dev mailing list