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

Allow reselection of node for Workload tree #300

Merged
merged 1 commit into from
Feb 6, 2017

Conversation

ZitaNemeckova
Copy link
Contributor

Services -> Workload -> choose any VM -> try to select selected node in tree to get back at list of VMs

Before: Cannot reselect node in tree
After: Can reselect node in tree.

Closes #291

@himdel please review :)

@ZitaNemeckova ZitaNemeckova changed the title Allow reselection of node for Worload tree Allow reselection of node for Workload tree Feb 6, 2017
@himdel
Copy link
Contributor

himdel commented Feb 6, 2017

@ZitaNemeckova aaah, much better, thanks! :)

But this is only one tree of two, there's also templates & images..

@ZitaNemeckova
Copy link
Contributor Author

@himdel Added second one.

@miq-bot
Copy link
Member

miq-bot commented Feb 6, 2017

Checked commit ZitaNemeckova@97ab63a with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 🍪

@himdel
Copy link
Contributor

himdel commented Feb 6, 2017

Aah, much more intuitive, thanks! :) Merging..

Adding enhancement, not bug because the problem was always present, it's just that now we have a nice way of fixing it.

@himdel himdel merged commit d8839d9 into ManageIQ:master Feb 6, 2017
@himdel himdel added this to the Sprint 54 Ending Feb 13, 2017 milestone Feb 6, 2017
@ZitaNemeckova ZitaNemeckova deleted the allow_reselect_workload branch September 12, 2017 14:29
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.

3 participants