Project

General

Profile

Bug #15935

updating autoheal on host from api doesn't seem to save value

Added by Justin Sherrill almost 6 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
High
Category:
Hosts
Target version:
Difficulty:
easy
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

To reproduce:

1. register a host with subscription-manager
2. go to Hosts > content hosts > click on the host
3. Try to change autoheal from true to false

Reload the page, it will be setill set to true

Associated revisions

Revision dda6c6f7 (diff)
Added by Justin Sherrill over 5 years ago

Fixes #15935 - allow for subscription facet updates

This change causes the accepts_nested_attributes_for to be used
from katello and not from foreman. This basically ignores the
:reject_if => :all_blank, which does not work for subscription_facet
as you might update :autoheal => false which would be blank

History

#1 Updated by Justin Sherrill almost 6 years ago

  • Category set to Hosts
  • Priority changed from Normal to High
  • Legacy Backlogs Release (now unused) set to 114
  • Difficulty set to easy

#2 Updated by The Foreman Bot almost 6 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Thomas McKay
  • Legacy Backlogs Release (now unused) deleted (114)
  • Pull request https://github.com/Katello/katello/pull/6256 added

#3 Updated by Justin Sherrill almost 6 years ago

  • Legacy Backlogs Release (now unused) set to 162

#4 Updated by Justin Sherrill over 5 years ago

  • Legacy Backlogs Release (now unused) changed from 162 to 197

#5 Updated by Justin Sherrill over 5 years ago

  • Status changed from Ready For Testing to Assigned
  • Assignee changed from Thomas McKay to Justin Sherrill
  • Pull request deleted (https://github.com/Katello/katello/pull/6256)

#6 Updated by The Foreman Bot over 5 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/6398 added

#7 Updated by Brad Buckingham over 5 years ago

  • Target version set to 129

#8 Updated by Justin Sherrill over 5 years ago

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

Also available in: Atom PDF