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

VmCloud volume detach: Let replace_right_cell turn off the spinner #4910

Merged

Conversation

mansam
Copy link
Contributor

@mansam mansam commented Nov 13, 2018

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1647695 by allowing replace_right_cell to be responsible for turning off the spinner when submitting a volume detach request from the VmCloud view.

@mansam
Copy link
Contributor Author

mansam commented Nov 13, 2018

@h-kataria

@mansam mansam force-pushed the fix-vm-cloud-volume-detach-spinner branch from 092dbbe to 8d5842c Compare November 14, 2018 20:13
@miq-bot
Copy link
Member

miq-bot commented Nov 14, 2018

Checked commit mansam@8d5842c with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

@mzazrivec mzazrivec self-assigned this Nov 19, 2018
@mzazrivec mzazrivec added this to the Sprint 99 Ending Nov 19, 2018 milestone Nov 19, 2018
@mzazrivec mzazrivec merged commit 2bb3222 into ManageIQ:master Nov 19, 2018
simaishi pushed a commit that referenced this pull request Nov 19, 2018
VmCloud volume detach: Let replace_right_cell turn off the spinner

(cherry picked from commit 2bb3222)

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

Hammer backport details:

$ git log -1
commit 9741632c2c5299ea67eeb316e23687ff5053dbff
Author: Milan Zázrivec <[email protected]>
Date:   Mon Nov 19 15:27:52 2018 +0100

    Merge pull request #4910 from mansam/fix-vm-cloud-volume-detach-spinner
    
    VmCloud volume detach: Let replace_right_cell turn off the spinner
    
    (cherry picked from commit 2bb3222f2a79fcb1892ac4e1745c86fd828da27e)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1647695

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