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 history button in vm infra page #12614

Merged
merged 1 commit into from
Nov 16, 2016

Conversation

ZitaNemeckova
Copy link
Contributor

Introduced by #11387

Compute -> Infrastructure -> Virtual Machine
Click on any VM
Click on its Snapshots
Click on another node in left tree
Click back button and choose one with Snapshots

All x_history_add_item that take id of VM/Template were corrected.

(aa is a VM, centos-6-v20150325 is parent node of VM.)
Before:
screen shot 2016-11-14 at 4 05 39 pm
Snapshots of parent node of VM.

After:
screen shot 2016-11-14 at 3 50 08 pm
Snapshots of VM.

This PR needs #12504 to work.

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

@miq-bot add_label ui, bug, blocker, euwe/yes

@himdel
Copy link
Contributor

himdel commented Nov 15, 2016

Looks good, but I'm seeing @sb[@sb[:active_accord]].present? ? @sb[@sb[:active_accord]] : x_node 5 times maybe you could put it in a method? ;) (I'm thinking something like x_right_node or x_node_right_cell or such..)

@ZitaNemeckova ZitaNemeckova force-pushed the fix_history_button_vm branch 2 times, most recently from dcc3eed to 4ac07e4 Compare November 15, 2016 15:27
@himdel
Copy link
Contributor

himdel commented Nov 15, 2016

LGTM, tested in the UI, fixes the bug 👍
Will merge when green..

@himdel himdel closed this Nov 15, 2016
@himdel himdel reopened this Nov 15, 2016
@himdel himdel closed this Nov 16, 2016
@himdel himdel reopened this Nov 16, 2016
@miq-bot
Copy link
Member

miq-bot commented Nov 16, 2016

Checked commit ZitaNemeckova@9b727fc with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
2 files checked, 0 offenses detected
Everything looks good. 🍰

@himdel himdel merged commit 804cf5e into ManageIQ:master Nov 16, 2016
@himdel himdel added this to the Sprint 50 Ending Dec 5, 2016 milestone Nov 16, 2016
chessbyte pushed a commit that referenced this pull request Nov 16, 2016
@chessbyte
Copy link
Member

Euwe Backport details:

$ git log -1
commit adbe1faf05a8cf6c6fd01a7a1e87827c7b5ba6e9
Author: Martin Hradil <himdel@seznam.cz>
Date:   Wed Nov 16 14:19:51 2016 +0200

    Merge pull request #12614 from ZitaNemeckova/fix_history_button_vm

    Fix history button in vm infra page
    (cherry picked from commit 804cf5e66e28d6370333148c4f2affc47a61de35)

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

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