Refactor #15410: Move activation_keys#subscriptions to subscriptions#index - Katello - Foreman

Refactor #15410

Move activation_keys#subscriptions to subscriptions#index

Added by Andrew Kofink about 3 years ago. Updated about 1 year ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

API

Target version:

Branch:

Difficulty:

Triaged:

Yes

Bugzilla link:

Pull request:

https://github.com/Katello/katello/pull/6124

Team Backlog:

Fixed in Releases:

Found in Releases:

Description

There's no need for a subscriptions action on the ActivationKeysController when Rails supports nested resources (i.e. /activation_keys/:activation_key_id/subscriptions). This change does not affect the API endpoint, just the controller and action that is hit.

Related issues

Associated revisions

Revision 2b0f4c5c (diff)
Added by Andrew Kofink about 3 years ago

Fixes #15410 - Update route for activation_key... (#6124)

Update routes for activation_key#subscriptions to point to
subsciprions#index with path
`/activation_keys/:activation_key_id/subscriptions`

This change is motivated by hammer-cli-katello bug #15272

History

#1 Updated by The Foreman Bot about 3 years ago

Status changed from New to Ready For Testing
Pull request https://github.com/Katello/katello/pull/6124 added

#2 Updated by Andrew Kofink about 3 years ago

Related to Bug #15272: 'hammer activation-key subscriptions' shows already removed subscriptions added

#3 Updated by Justin Sherrill about 3 years ago

Legacy Backlogs Release (now unused) set to 162

#4 Updated by Anonymous about 3 years ago

Status changed from Ready For Testing to Closed
% Done changed from 0 to 100

Applied in changeset katello|2b0f4c5cdad6048108f7fc43116dcb67c5d0f6d5.

#5 Updated by Daniel Lobato Garcia over 2 years ago

Bugzilla link set to 1406076

Also available in: Atom PDF