Project

General

Profile

Actions

Bug #8217

closed

Matchers/overrides have no clear grouping, mess of input boxes

Added by Dominic Cleal over 9 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

The overrides section of the smart class parameter page has become a bit of a maze of checkboxes and input boxes, and needs tidying up.

The main problems that I see are:

1. "Global" configuration for the behaviour of overrides (merging, "avoid duplicates", order) should be more separate to the list of matchers.
2. No separation between individual matchers, so what the "Use Puppet default" box applies to is unclear. Other forms in the app use horizontal rules to separate individual items.


Files


Related issues 3 (1 open2 closed)

Related to Foreman - Feature #3260: Allow puppet parameters to have no default value, but overrideableClosedOri Rabin10/15/2013Actions
Related to Foreman - Feature #3309: Support deep merging of hash structures in smart class parametersClosedOri Rabin10/17/2013Actions
Blocks Foreman - Tracker #4470: Usability of parameters and overridesNew

Actions
Actions #2

Updated by Dominic Cleal over 9 years ago

  • Related to Feature #3260: Allow puppet parameters to have no default value, but overrideable added
Actions #3

Updated by Dominic Cleal over 9 years ago

  • Related to Feature #3309: Support deep merging of hash structures in smart class parameters added
Actions #4

Updated by Dominic Cleal over 9 years ago

  • Blocks Tracker #4470: Usability of parameters and overrides added
Actions #5

Updated by Ori Rabin over 9 years ago

  • Status changed from New to Assigned
  • Assignee set to Ori Rabin
Actions #6

Updated by Dominic Cleal over 9 years ago

3. "Use Puppet default" is below the default value, but above the override value text boxes

Actions #7

Updated by The Foreman Bot over 9 years ago

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

Updated by Ori Rabin almost 9 years ago

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

Updated by Dominic Cleal almost 9 years ago

  • translation missing: en.field_release set to 35
Actions

Also available in: Atom PDF