Project

General

Profile

Actions
Copy link

Bug #11639

closed

Package group deprecation method needs to be removed

Added by John Mitsch over 9 years ago. Updated over 6 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
John Mitsch
Category:
API
Target version:
Katello 2.4.0
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

This line https://github.com/Katello/katello/pull/5380/files#diff-b35c563e3598aa9e5f1a072f1cdeaf0bR8 in the API deprecation warning was removed by this PR https://github.com/Katello/katello/pull/5341/files#diff-b35c563e3598aa9e5f1a072f1cdeaf0bL8

The warning needs to be added back

Actions
Copy link
 #1

Updated by John Mitsch over 9 years ago

  • Translation missing: en.field_release set to 77
Actions
Copy link
 #2

Updated by John Mitsch over 9 years ago

not sure if this has to happen on second thought

Actions
Copy link
 #3

Updated by John Mitsch over 9 years ago

  • Translation missing: en.field_release changed from 77 to 70

UPDATE: just the deprecation function has to be removed https://github.com/Katello/katello/blob/master/app/controllers/katello/api/v2/content_view_filters_controller.rb#L73

The api endpoint that the deprecation warning is being called for is currently removed in development, but was part of 2.3. the deprecation function is not useful anymore.

Actions
Copy link
 #4

Updated by John Mitsch over 9 years ago

  • Subject changed from Package group deprecation warning was removed to Package group deprecation function needs to be removed
Actions
Copy link
 #5

Updated by John Mitsch over 9 years ago

  • Subject changed from Package group deprecation function needs to be removed to Package group deprecation method needs to be removed
Actions
Copy link
 #6

Updated by Eric Helms over 9 years ago

  • Category set to API
  • Assignee set to John Mitsch
  • Triaged changed from No to Yes
Actions
Copy link
 #7

Updated by The Foreman Bot over 9 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2673 added
  • Pull request deleted ()
Actions
Copy link
 #8

Updated by Dominic Cleal over 9 years ago

  • Status changed from Ready For Testing to New
  • Pull request added
  • Pull request deleted (https://github.com/theforeman/foreman/pull/2673)
Actions
Copy link
 #9

Updated by John Mitsch over 9 years ago

  • Status changed from New to Rejected

This isn't necessary because the warning was added in 2.3 and will be removed in 2.4

Actions
Copy link

Also available in: Atom PDF