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 missing tree view in Datastore Clusters accordion #129

Conversation

hstastna
Copy link

@hstastna hstastna commented Jan 11, 2017

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

Fix missing tree view in Datastore Clusters accordion in
Compute -> Infrastructure -> Datastores -> Datastore Clusters
by removing Advanced searchbox which should not be available
when reaching Datastore Clusters accordion. Advanced search
should only be available on Datastores accordion that displays
filters in tree.

This bug is fixed by adding a condition to display_adv_searchbox method
in storage controller which has a role to hide/show advanced searchbox.
Originally it was designed to show advanced searchbox all the time in
Compute -> Infrastructure -> Datastores .

This bug was also mentioned in ManageIQ/manageiq#13140

Before:
datastore_clusters

After:
datastore_clusters2

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

Fix missing tree view in Datastore Clusters accordion in
Compute -> Infrastructure -> Datastores -> Datastore Clusters
by removing Advanced searchbox which should not be available
when reaching Datastore Clusters accordion. Advanced search
should only be available on Datastores accordion that displays
filters in tree.
@hstastna hstastna force-pushed the Filter_Datastore_Clusters_missing_tree_view branch from 94cef7d to 6c722bc Compare January 16, 2017 14:06
@miq-bot
Copy link
Member

miq-bot commented Jan 16, 2017

Checked commit hstastna@6c722bc with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
1 file checked, 0 offenses detected
Everything looks good. 🍰

@mzazrivec mzazrivec self-assigned this Jan 16, 2017
@mzazrivec mzazrivec added this to the Sprint 52 Ending Jan 16, 2017 milestone Jan 16, 2017
@mzazrivec mzazrivec merged commit 3f491c1 into ManageIQ:master Jan 16, 2017
@simaishi
Copy link
Contributor

Euwe backport (to manageiq repo) details:

$ git log -1
commit f18ccbf2b299f90193d910b2eb67cf0093f8486d
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Mon Jan 16 15:27:04 2017 +0100

    Merge pull request #129 from hstastna/Filter_Datastore_Clusters_missing_tree_view
    
    Fix missing tree view in Datastore Clusters accordion
    (cherry picked from commit 3f491c18747c1adbf87fcedd7d780dd907601097)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1414872

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.

4 participants