Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Save new dialog value & other UI issues fixed #1330

Merged
merged 1 commit into from
May 16, 2017

Conversation

h-kataria
Copy link
Contributor

Fixed saving of new dialog name, reset 'Hosts' field when playbook is selected on Retirement tab during edit and fixed issues related to initialization of Escalate Privilege switch

https://bugzilla.redhat.com/show_bug.cgi?id=1448228

@lgalis @bzwei please test.

Please see steps to recreate issues with Escalate Privilege switch & hosts field below:
To see Escalate Privilege switch in a weird state:

  1. change value of escalate privilege field on a catalog item that was added prior to add of escalate privilege field, and press reset button
  2. Save catalog item without playbook on retirement tab, then go to edit the same catalog item, and select playbook field on retirement tab.

Hosts field: Save a catalog item without a playbook on retirement tab. Go to edit the same catalog item, value of Hosts text box is empty, after selecting Playbook it should initialize Hosts text field.

Fixed saving of new dialog name, reset 'Hosts' field when playbook is selected on Retirement tab during edit and fixed issues related to initialization of Escalate Privilege switch

https://bugzilla.redhat.com/show_bug.cgi?id=1448228
@miq-bot
Copy link
Member

miq-bot commented May 10, 2017

Checked commit h-kataria@4f29b13 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@lgalis
Copy link
Contributor

lgalis commented May 11, 2017

Looks good. Tested in the ui for both the uninitialized escalation flag as well as the Host field in the retirement tab.

@dclarizio dclarizio merged commit 62c2b33 into ManageIQ:master May 16, 2017
@dclarizio dclarizio added this to the Sprint 61 Ending May 22, 2017 milestone May 16, 2017
@h-kataria h-kataria deleted the new_dialog_name_fix branch May 18, 2017 17:29
simaishi pushed a commit that referenced this pull request Jun 8, 2017
Save new dialog value & other UI issues fixed
(cherry picked from commit 62c2b33)

https://bugzilla.redhat.com/show_bug.cgi?id=1459962
@simaishi
Copy link
Contributor

simaishi commented Jun 8, 2017

Fine backport details:

$ git log -1
commit 3ceefe6a55df51a6c01c00f3a53453a055b723c0
Author: Dan Clarizio <[email protected]>
Date:   Tue May 16 14:38:03 2017 -0700

    Merge pull request #1330 from h-kataria/new_dialog_name_fix
    
    Save new dialog value & other UI issues fixed
    (cherry picked from commit 62c2b3390681033eeb82a5db9914621dc98826cf)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1459962

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants