-
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
MiqRequest: fix listing of affected items. #2910
MiqRequest: fix listing of affected items. #2910
Conversation
Passing of :selected_ids throught get_view(..) is not implemented. We shall implement that after Gaprindashvili. For now pass record ids throught an instance variable.
2b47a60
to
5ad58b6
Compare
5ad58b6
to
237c543
Compare
Ping @tumido : you broke it ;-) Review, please? |
Some comments on commits martinpovolny/manageiq-ui-classic@9568587~...237c543 spec/controllers/miq_request_controller_spec.rb
|
Checked commits martinpovolny/manageiq-ui-classic@9568587~...237c543 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 |
MiqRequest: fix listing of affected items. (cherry picked from commit 8a01085) https://bugzilla.redhat.com/show_bug.cgi?id=1520660
Gaprindashvili backport details:
|
Passing of :selected_ids throught get_view(..) is not implemented.
We shall implement that after Gaprindashvili.
For now pass record ids throught an instance variable.
Fixes:
https://bugzilla.redhat.com/show_bug.cgi?id=1513303
TODO:
write a speccreate TODO item for passing the selected_records throughget_view