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

Don't allow adding another svc bundle into an existing bundle #5221

Conversation

mzazrivec
Copy link
Contributor

@JPrause
Copy link
Member

JPrause commented Feb 11, 2019

@mzazrivec who can help review this PR. We need this for the hammer build.

@h-kataria h-kataria self-assigned this Feb 13, 2019
@h-kataria
Copy link
Contributor

@mzazrivec can you also add spec test for this change, otherwise change looks good.

@mzazrivec mzazrivec force-pushed the dont_show_another_bundle_when_editing_bundle branch from 436b774 to 60f6990 Compare February 14, 2019 10:58
@mzazrivec
Copy link
Contributor Author

@h-kataria test added.

@miq-bot
Copy link
Member

miq-bot commented Feb 14, 2019

Checked commits mzazrivec/manageiq-ui-classic@b4fbee4~...60f6990 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@h-kataria h-kataria added this to the Sprint 105 Ending Feb 18, 2019 milestone Feb 14, 2019
@h-kataria h-kataria merged commit 60b3dff into ManageIQ:master Feb 14, 2019
@mzazrivec mzazrivec deleted the dont_show_another_bundle_when_editing_bundle branch February 14, 2019 17:56
simaishi pushed a commit that referenced this pull request Feb 18, 2019
…_editing_bundle

Don't allow adding another svc bundle into an existing bundle

(cherry picked from commit 60b3dff)

https://bugzilla.redhat.com/show_bug.cgi?id=1678368
@simaishi
Copy link
Contributor

Hammer backport details:

$ git log -1
commit 7369dc67da4c4b2dc0bd9c2a880bfcda2b7beeb8
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Thu Feb 14 12:44:41 2019 -0500

    Merge pull request #5221 from mzazrivec/dont_show_another_bundle_when_editing_bundle
    
    Don't allow adding another svc bundle into an existing bundle
    
    (cherry picked from commit 60b3dffa9e28c41299b68073523678531305ac73)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1678368

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.

5 participants