-
Notifications
You must be signed in to change notification settings - Fork 360
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 parent_id being a number, not string in JS #4399
Conversation
app/controllers/vm_common.rb
@himdel The CI failure seems related. Also, could you please fix the rubocop warning above? Thanks. |
Thanks, should be fixed now :) |
Still failing in |
Checked commits https://github.com/himdel/manageiq-ui-classic/compare/f49bafae5cf164cdbb5e7447ed5c115ca4a7e26c~...bbadd37504e0f08dc169eaeab61cc3a8d51a1098 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
Not anymore :) whew :) |
issue: #1405 |
@miq-bot add_label blocker |
Fix parent_id being a number, not string in JS (cherry picked from commit f6c97ab) https://bugzilla.redhat.com/show_bug.cgi?id=1613333
Gaprindashvili backport details:
|
Related to #3243 and #4397 and #2902.
Trying to make sure
parent_id
always ends up as a string in JS, not a number.https://bugzilla.redhat.com/show_bug.cgi?id=1610353
Cc @skateman