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

Specify a target and resource action when retrieving a service dialog #231

Merged
merged 1 commit into from Nov 30, 2017

Conversation

jntullo
Copy link

@jntullo jntullo commented Nov 29, 2017

Fetching the content from a resource sometimes requires both a target and resource_action (ie for custom buttons), where we are currently passing in nil values. By accepting resource_action_id, target_type, and target_id, we are able to use resource search to pass in the applicable values.

Please verify @eclarizio

@miq-bot add_label bug, blocker
@miq-bot add_label gaprindashvili/yes

https://bugzilla.redhat.com/show_bug.cgi?id=1518390

Fetching the content from a resource sometimes requires both a target and resource_action, where we are currently passing in nil values. By accepting resource_action_id, target_type, and target_id, we are able to use resource search to pass in the applicable values.

https://bugzilla.redhat.com/show_bug.cgi?id=1518390
Copy link
Member

@eclarizio eclarizio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, thanks for this 👍

@miq-bot
Copy link
Member

miq-bot commented Nov 29, 2017

Checked commit jntullo@b100aa7 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 1 offense detected

app/controllers/api/service_dialogs_controller.rb

Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@gtanzillo gtanzillo self-assigned this Nov 30, 2017
@gtanzillo gtanzillo added this to the Sprint 75 Ending Dec 11, 2017 milestone Nov 30, 2017
@gtanzillo gtanzillo merged commit a70e73d into ManageIQ:master Nov 30, 2017
simaishi pushed a commit that referenced this pull request Dec 4, 2017
Specify a target and resource action when retrieving a service dialog
(cherry picked from commit a70e73d)
@simaishi
Copy link
Contributor

simaishi commented Dec 4, 2017

Gaprindashvili backport details:

$ git log -1
commit 82d437d673ab05dd4ad4c8d74e7312c1506a42f0
Author: Gregg Tanzillo <gtanzill@redhat.com>
Date:   Thu Nov 30 13:45:55 2017 -0500

    Merge pull request #231 from jntullo/dialog_content_targets
    
    Specify a target and resource action when retrieving a service dialog
    (cherry picked from commit a70e73d01c8388690554ed2563dda894c78efe4f)

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.

None yet

5 participants