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

Fix Default tab error indicator for OpenStack Infra #1341

Merged

Conversation

AparnaKarve
Copy link
Contributor

@AparnaKarve AparnaKarve commented May 12, 2017

No error indicator is displayed on the Default tab when OpenStack Infra is selected in the New Infra Provider form
Also New Cloud Provider form shows 3 tabs right off the bat even though Type is not selected

Before (New Infra Provider Form with OpenStack selected and no error indicator on Default tab) --
screen shot 2017-05-12 at 2 51 21 pm

After (New Infra Provider Form with OpenStack selected and error indicator visible on Default tab) --
screen shot 2017-05-12 at 2 49 27 pm


Before (New Cloud Provider Form with no Type selected and 3 tabs visible) --
screen shot 2017-05-12 at 2 50 53 pm

After (New Cloud Provider Form with no Type selected and no tabs visible) --
screen shot 2017-05-12 at 2 50 03 pm

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

@AparnaKarve
Copy link
Contributor Author

@miq-bot add-label bug,fine/yes

@mzazrivec
Copy link
Contributor

On the infra provider page:

  • select openstack infra, the error indicator is there
  • select vmware
  • select openstack infra again, the error indicator is not there

@AparnaKarve AparnaKarve force-pushed the fix_new_ems_form_error_indicator branch from 736df8f to 8ec313c Compare May 15, 2017 17:42
@miq-bot
Copy link
Member

miq-bot commented May 15, 2017

Checked commits AparnaKarve/manageiq-ui-classic@73efadc~...8ec313c with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@AparnaKarve
Copy link
Contributor Author

@mzazrivec Thanks!
I myself was not convinced about the changes in 9188efa since that was just helping alleviate the issue but was not exactly the root cause.
The root cause, believe it or not, was the the equality statement in 8ec313c == Vs. ===

FYI, The below comparison between null and undefined came into play for this particular use case.

null == undefined //true
null === undefined //false

@mzazrivec mzazrivec added this to the Sprint 61 Ending May 22, 2017 milestone May 16, 2017
@mzazrivec mzazrivec merged commit 55e6006 into ManageIQ:master May 16, 2017
simaishi pushed a commit that referenced this pull request Jun 8, 2017
…cator

Fix Default tab error indicator for OpenStack Infra
(cherry picked from commit 55e6006)

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

simaishi commented Jun 8, 2017

Fine backport details:

$ git log -1
commit 7257c9f1e200cfeb17074c1a011c63bd8f1e9d07
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Tue May 16 13:56:51 2017 +0200

    Merge pull request #1341 from AparnaKarve/fix_new_ems_form_error_indicator
    
    Fix Default tab error indicator for OpenStack Infra
    (cherry picked from commit 55e60069140bcf01b34c9d09903c4f3bd53cc194)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1459923

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