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 storage_controller to show custom buttons #3629

Merged
merged 1 commit into from
Mar 19, 2018

Conversation

ZitaNemeckova
Copy link
Contributor

@ZitaNemeckova ZitaNemeckova commented Mar 15, 2018

Closes #996

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

Steps to reproduce:

  • Go to Automation > Automate > Customization, Buttons accordion
  • pick Datastore in the tree
  • create a group, and a custom button
  • then go to Compute > Infrastructure > Datastores
  • click on a datastore
    Before:

screen shot 2018-03-15 at 2 26 24 pm

screen shot 2018-03-15 at 2 26 16 pm

After:
screen shot 2018-03-15 at 2 17 45 pm
screen shot 2018-03-15 at 2 17 31 pm

It cannot rely on @lastaction == "show_list" because it's always "explorer". Introduced in ManageIQ/manageiq#11793

@miq-bot add_label bug, toolbars, gaprindashvili/yes

Closes ManageIQ#996

It cannot rely on @lastaction == 'show_list' because it's always 'explorer'. Introduced in ManageIQ/manageiq#11793
@ZitaNemeckova ZitaNemeckova force-pushed the datastore_custom_buttons branch from a7281e1 to a1e490b Compare March 15, 2018 13:41
@miq-bot
Copy link
Member

miq-bot commented Mar 15, 2018

Checked commit ZitaNemeckova@a1e490b with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

@mzazrivec mzazrivec self-assigned this Mar 19, 2018
@mzazrivec mzazrivec added this to the Sprint 82 Ending Mar 26, 2018 milestone Mar 19, 2018
@mzazrivec mzazrivec merged commit 3ccf3bd into ManageIQ:master Mar 19, 2018
@simaishi
Copy link
Contributor

@ZitaNemeckova Please add BZ link.

@ZitaNemeckova
Copy link
Contributor Author

@simaishi Added. Thanks :)

@ZitaNemeckova ZitaNemeckova deleted the datastore_custom_buttons branch March 20, 2018 11:44
@ZitaNemeckova
Copy link
Contributor Author

@miq-bot add_label euwe/yes, fine/yes

simaishi pushed a commit that referenced this pull request Mar 20, 2018
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit c8290fe6b6179a0edb47094b7bddcd5af762ea4b
Author: Milan Zázrivec <[email protected]>
Date:   Mon Mar 19 09:52:09 2018 +0100

    Merge pull request #3629 from ZitaNemeckova/datastore_custom_buttons
    
    Fix storage_controller to show custom buttons
    (cherry picked from commit 3ccf3bd8521adb05958c404bbb4eec38ebd0ab5d)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1558544

@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit 5099e23983363c850f58eeef016e5e2fdebb2b4c
Author: Milan Zázrivec <[email protected]>
Date:   Mon Mar 19 09:52:09 2018 +0100

    Merge pull request #3629 from ZitaNemeckova/datastore_custom_buttons
    
    Fix storage_controller to show custom buttons
    (cherry picked from commit 3ccf3bd8521adb05958c404bbb4eec38ebd0ab5d)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1565403

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