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

Return correct subcollection actions #243

Merged
merged 1 commit into from
Dec 11, 2017
Merged

Conversation

jntullo
Copy link

@jntullo jntullo commented Dec 11, 2017

Right now, sub collection actions that are specified on a specific collection (i.e., :generic_object_subcollection_actions: specified on :services: collection) are not taking precedence to those sub collection actions defined on the collection itself (i.e. :subcollection_actions: defined under :generic_objects:). This is causing incorrect actions to be returned, causing the users to believe that they have actions that they do not, such as :create: for the generic objects sub collection on services.

This is a reminder to myself that I should probably get around to fixing up #119 ( new year's resolution 😉 )

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

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

Right now, sub collection actions that are specified on a specific collection (i.e., :generic_object_subcollection_actions: specified on :services: collection) are not taking precedence to those sub collection actions defined on the collection itself (i.e. :subcollection_actions: defined under :generic_objects:). This is causing incorrect actions to be returned, causing the users to believe that they have actions that they do not, such as :create: for the generic objects sub collection on services.

https://bugzilla.redhat.com/show_bug.cgi?id=1518833
@miq-bot
Copy link
Member

miq-bot commented Dec 11, 2017

Checked commit jntullo@7b13b36 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. ⭐

@abellotti abellotti self-requested a review December 11, 2017 20:45
@abellotti abellotti self-assigned this Dec 11, 2017
@abellotti abellotti added this to the Sprint 75 Ending Dec 11, 2017 milestone Dec 11, 2017
@abellotti
Copy link
Member

👍 @jntullo Thanks for fixing this.

@abellotti abellotti merged commit d5cd683 into ManageIQ:master Dec 11, 2017
simaishi pushed a commit that referenced this pull request Dec 12, 2017
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 4cad0b375e33232694a9714c297204d8bcd56a6c
Author: Alberto Bellotti <abellotti@users.noreply.github.com>
Date:   Mon Dec 11 15:46:16 2017 -0500

    Merge pull request #243 from jntullo/bz_1518833
    
    Return correct subcollection actions
    (cherry picked from commit d5cd683270a9bbd2b3daf1af789162919dcbcb74)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1525049

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