Project

General

Profile

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:
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

Related to Katello - Bug #15272: 'hammer activation-key subscriptions' shows already removed subscriptionsClosed2016-06-02

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

#5 Updated by Daniel Lobato Garcia over 2 years ago

  • Bugzilla link set to 1406076

Also available in: Atom PDF