-
Notifications
You must be signed in to change notification settings - Fork 357
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 returning back to the previous controller, after clicking on Back button and to cancel policy sim properly #5839
Conversation
@miq-bot add_label hammer/yes, bug |
a00132d
to
e7ccc5d
Compare
e7ccc5d
to
3d488bc
Compare
3d488bc
to
ab2a316
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…nges" This reverts commit ed58a5c.
…ore" This reverts commit 2e6644b.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1717495 Also display quadicon of template which did not show up in policy simulation page when running against VMs and Templates.
ab2a316
to
d4de8de
Compare
Checked commits hstastna/manageiq-ui-classic@331b320~...d4de8de with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 |
@miq-bot add_label ivanchuk/yes |
Allow returning back to the previous controller, after clicking on Back button and to cancel policy sim properly (cherry picked from commit aa7b10b) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1717495
Ivanchuk backport details:
|
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1717495
When clicking on quadion of a VM or Template while running policy simulation, the actual controller was changed to
VmOrTemplate
. After clicking on Back button, it remained the same. This was causing unexpected behavior later in the next steps, many variables not set properly, not able to cancel policy sim properly... To prevent this controller's 'change' #5580 was created.The solution worked well except the case that user chose some items from the list which were both VMs AND Templates, for policy sim, in Compute > Infra > Virtual Machines > VMs & Templates accordion. The problem was missing quadicon(s) of Template in policy simulation.
So little bit different approach is good to implement: going 'back' to the previous controller after clicking on Back button (instead of the same controller all the time). So I specified the controller name for getting proper url for
onclick
for appropriate Back button. By usingmodel_for_vm
method we get a proper model of the@record
, and withcontroller_for_vm
we get a proper controller name.Before:
After: