[openstreetmap/openstreetmap-website] API element resources - full paths (PR #5620)

Anton Khorev notifications at github.com
Wed Feb 5 18:13:07 UTC 2025


`full` is not a standard action name, but it's also not really a nested resource. Here I'm mapping them to `show` action with a parameter. A similar thing is done with changeset discussions where the discussion/comment elements are extra things added to the usual `show` output if a parameter is set.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Map 'full' to api way show action
  * Map 'full' to api relation show action

-- File Changes --

    M app/abilities/api_ability.rb (4)
    M app/controllers/api/relations_controller.rb (129)
    M app/controllers/api/ways_controller.rb (35)
    D app/views/api/relations/full.json.jbuilder (7)
    D app/views/api/relations/full.xml.builder (7)
    M app/views/api/relations/show.json.jbuilder (4)
    M app/views/api/relations/show.xml.builder (4)
    D app/views/api/ways/full.json.jbuilder (6)
    D app/views/api/ways/full.xml.builder (6)
    M app/views/api/ways/show.json.jbuilder (1)
    M app/views/api/ways/show.xml.builder (1)
    M config/routes.rb (14)
    M test/controllers/api/relations_controller_test.rb (38)
    M test/controllers/api/ways_controller_test.rb (44)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/5620 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250205/57474ddf/attachment-0001.htm>


More information about the rails-dev mailing list