Project

General

Profile

Bug #34747

Adding the host parameters clears the values of the prior line

Added by Ron Lavi 12 months ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Ron Lavi
Category:
JavaScript stack
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
2056469
Pull request:
https://github.com/theforeman/foreman/pull/9180, https://github.com/theforeman/foreman/pull/9197, https://github.com/theforeman/foreman/pull/9196
Fixed in Releases:
3.1.3, 3.2.1, 3.3.0
Found in Releases:
Red Hat JIRA:

Description

https://community.theforeman.org/t/foreman-ui-propably-bug-in-js-in-3-2-0/27995

When you try to override host global parameters override parameter is cleared when you click to override secondone.

Click on host, then Edit → Parameters. You must have some global parameters there.
Click on override (works fine for firstone).
Click on override for second one → It cleared first override

Expected outcome:
When clicking on override it should just add new parameter with default values, not cleared previous override.

Associated revisions

Revision 99d67b17 (diff)
Added by Ron Lavi 12 months ago

Fixes #34747 - override_param should consider direction

after #8667 (RM32013) improved the UX with the ability to prepend items,
the `override_param` function should have been also updated
to consider the direction of the added items.

History

#1 Updated by The Foreman Bot 12 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/9180 added

#2 Updated by The Foreman Bot 12 months ago

  • Fixed in Releases 3.3.0 added

#3 Updated by Ron Lavi 12 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/9196 added

#5 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/9197 added

#6 Updated by Amit Upadhye 11 months ago

  • Category set to JavaScript stack

#7 Updated by Oleh Fedorenko 11 months ago

  • Bugzilla link set to 2056469

#8 Updated by The Foreman Bot 11 months ago

  • Fixed in Releases 3.2.1 added

#9 Updated by The Foreman Bot 11 months ago

  • Fixed in Releases 3.1.3 added

#10 Updated by Amit Upadhye 10 months ago

  • Subject changed from Adding host parameters clears the values of the prior line to Adding the host parameters clears the values of the prior line

Also available in: Atom PDF