Skip to content
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

Merged

Conversation

martinpovolny
Copy link
Member

@martinpovolny martinpovolny commented Dec 2, 2017

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 spec
  • create TODO item for passing the selected_records through get_view

@martinpovolny martinpovolny changed the title MiqRequest: fix listing of affected items. [WIP] MiqRequest: fix listing of affected items. Dec 2, 2017
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.
@martinpovolny martinpovolny force-pushed the miq_request_details_grid_fix branch 2 times, most recently from 2b47a60 to 5ad58b6 Compare December 2, 2017 12:27
@martinpovolny
Copy link
Member Author

Ping @tumido : you broke it ;-) Review, please?
@mzazrivec, @karelhala : can you review this one?

@martinpovolny martinpovolny changed the title [WIP] MiqRequest: fix listing of affected items. MiqRequest: fix listing of affected items. Dec 2, 2017
@martinpovolny martinpovolny removed the wip label Dec 2, 2017
@miq-bot
Copy link
Member

miq-bot commented Dec 2, 2017

Some comments on commits martinpovolny/manageiq-ui-classic@9568587~...237c543

spec/controllers/miq_request_controller_spec.rb

  • ⚠️ - 250 - Detected expect_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.

@miq-bot
Copy link
Member

miq-bot commented Dec 2, 2017

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
3 files checked, 0 offenses detected
Everything looks fine. ⭐

@mzazrivec mzazrivec self-assigned this Dec 4, 2017
@mzazrivec mzazrivec added this to the Sprint 75 Ending Dec 11, 2017 milestone Dec 4, 2017
@mzazrivec mzazrivec merged commit 8a01085 into ManageIQ:master Dec 4, 2017
simaishi pushed a commit that referenced this pull request Dec 4, 2017
@simaishi
Copy link
Contributor

simaishi commented Dec 4, 2017

Gaprindashvili backport details:

$ git log -1
commit cbb3b70c7ad9f7a496965b062766a51b9a74965a
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Mon Dec 4 09:06:14 2017 +0100

    Merge pull request #2910 from martinpovolny/miq_request_details_grid_fix
    
    MiqRequest: fix listing of affected items.
    (cherry picked from commit 8a01085ee0b5d30e0bd81f9f3badbfa9f3eb5a61)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1520660

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants