-
Notifications
You must be signed in to change notification settings - Fork 900
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
Conversation
Looks good, but I'm seeing |
dcc3eed
to
4ac07e4
Compare
LGTM, tested in the UI, fixes the bug 👍 |
4ac07e4
to
9b727fc
Compare
Checked commit ZitaNemeckova@9b727fc with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1 |
Fix history button in vm infra page (cherry picked from commit 804cf5e) https://bugzilla.redhat.com/show_bug.cgi?id=1393708
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 |
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:
Snapshots of parent node of VM.
After:

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