[openstreetmap/openstreetmap-website] Rename api OldController to OldElementsController (PR #4642)
Anton Khorev
notifications at github.com
Tue Apr 2 13:18:05 UTC 2024
#3715 adds `ElementsController`. When I originally did it, I didn't notice that a similar superclass already exists for old elements because it's not named after elements.
The name "OldController" is not specific enough because there are other "old" things besides old elements like old tags or old relation members.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/4642
-- Commit Summary --
* Rename api OldController to OldElementsController
-- File Changes --
R app/controllers/api/old_elements_controller.rb (2)
M app/controllers/api/old_nodes_controller.rb (2)
M app/controllers/api/old_relations_controller.rb (2)
M app/controllers/api/old_ways_controller.rb (2)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/4642.patch
https://github.com/openstreetmap/openstreetmap-website/pull/4642.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4642
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4642 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240402/ff238718/attachment.htm>
More information about the rails-dev
mailing list