Refs #5539 Although I've renamed the actions, I've left the urls alone for now. Further refactoring might include things like:
* Reconsider whether the embedded html source, data download and image export functionality should all really be in the same controller * Make resourceful routes for whatever gets decided * Remove hardcoded form target urls from forms and use route helpers instead You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6551 -- Commit Summary -- * Rename export controller actions -- File Changes -- M .rubocop.yml (1) M app/abilities/ability.rb (2) M app/controllers/export_controller.rb (12) R app/views/export/show.html.erb (0) M app/views/site/export.html.erb (2) M config/routes.rb (4) M test/controllers/export_controller_test.rb (26) -- Patch Links -- https://github.com/openstreetmap/openstreetmap-website/pull/6551.patch https://github.com/openstreetmap/openstreetmap-website/pull/6551.diff -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6551 You are receiving this because you are subscribed to this thread. Message ID: <openstreetmap/openstreetmap-website/pull/[email protected]>
_______________________________________________ rails-dev mailing list [email protected] https://lists.openstreetmap.org/listinfo/rails-dev
