-
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
Remove catching flash messages in report_data_controller.js #4789
Conversation
@miq-bot add_label gaprindashvili/no, hammer/yes, bug, gtls |
https://bugzilla.redhat.com/show_bug.cgi?id=1639213 * oneMessage.msg is actually undefined (it should be oneMessage.message), it probably never works
Checked commit rvsia@85cc039 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
I tried this PR and it fixes also my BZ https://bugzilla.redhat.com/show_bug.cgi?id=1611612. 👍 from my side |
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.
Tested in UI, fixes the issue 👍
Not seeing any indications of |
Remove catching flash messages in report_data_controller.js (cherry picked from commit 888aa60) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1639213
Hammer backport details:
|
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1639213
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1611612
Steps to Reproduce:
or Compute -> Clouds -> Instances
or Lifecycle -> Provision Instances
Actual results:
Two success messages appear, one without text
Expected results:
One success message appears
Before / After
NOTES
The
oneMessage.msg
( here , introduced by #1245 ) is undefined, so it's causing empty messages. It should beoneMessage.message
, but then it's causing double messages.I don't know if
msg
is used somewhere else, but I checked issues from #1273 (which should be fixed by this piece of code) after I had removed the function and they are fixed anyway. So I think the function actually doesn't work right now. Maybe there was another change which fixed all these thing. I don't know.@karelhala @himdel @skateman Pinging you, because you were involved in that PR.