Refactor #6287
Convert remaining v2 API controllers to use Api::V2::ApiController
Pull request:
Fixed in Releases:
Found in Releases:
Description
These controllers still need to be converted to v2 proper.
[walden@wraines-sandbox katello (master)]$ grep -rl "Api::V1" app/controllers/katello/api/v2 | sort app/controllers/katello/api/v2/about_controller.rb app/controllers/katello/api/v2/custom_info_controller.rb app/controllers/katello/api/v2/distributors_controller.rb app/controllers/katello/api/v2/host_collection_errata_controller.rb app/controllers/katello/api/v2/host_collection_packages_controller.rb app/controllers/katello/api/v2/organization_default_info_controller.rb app/controllers/katello/api/v2/permissions_controller.rb app/controllers/katello/api/v2/role_ldap_groups_controller.rb app/controllers/katello/api/v2/roles_controller.rb app/controllers/katello/api/v2/root_controller.rb app/controllers/katello/api/v2/uebercerts_controller.rb
Currently they are all still using v1. We can split these out into individual issues if needed. As part of this we should also remove the remaining v1 code.
Associated revisions
History
#1
Updated by Walden Raines almost 8 years ago
- Subject changed from Convert remaining v2 controllers to to Convert remaining v2 API controllers to use Api::V2::ApiController
#2
Updated by Walden Raines almost 8 years ago
In addition to those above the following v1 API controllers have no v2 counterpart:
app/controllers/katello/api/v1/candlepin_proxies_controller.rb app/controllers/katello/api/v1/crls_controller.rb app/controllers/katello/api/v1/errors_controller.rb app/controllers/katello/api/v1/filters_controller.rb
These should also either be converted or removed.
#3
Updated by Eric Helms over 7 years ago
- Legacy Backlogs Release (now unused) set to 14
#4
Updated by The Foreman Bot over 7 years ago
- Status changed from New to Ready For Testing
- Target version set to 55
- Pull request https://github.com/Katello/katello/pull/4630 added
#5
Updated by Eric Helms over 7 years ago
- Assignee set to Eric Helms
#6
Updated by Eric Helms over 7 years ago
- Target version changed from 55 to 56
#7
Updated by Eric Helms over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset katello|5006f51b5841efba7786f5f4779130b7c42a1c9c.
Fixes #6287: Remove V1 API entirely.
This commit removes the existence of the V1 API by moving some controllers
to pure V2 API controllers and removing other, unused API controllers
and their corresponding tests.