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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add product setting default for allowing API service ordering #18029

Merged
merged 1 commit into from
Oct 19, 2018

Conversation

eclarizio
Copy link
Member

As an addition to this PR, @bdunne suggested I change the product setting to a positive name and default the value to true.

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

I know there's not much to "review" here, but @bdunne, can you give it a look? 馃憖

/cc @tinaafitz

@miq-bot add_label bug

@miq-bot
Copy link
Member

miq-bot commented Oct 15, 2018

This pull request is not mergeable. Please rebase and repush.

@miq-bot
Copy link
Member

miq-bot commented Oct 15, 2018

Checked commit eclarizio@5517c40 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 馃嵃

@JPrause
Copy link
Member

JPrause commented Oct 18, 2018

@miq-bot add_label blocker

@JPrause
Copy link
Member

JPrause commented Oct 18, 2018

@eclarizio is this will be able to be backported, can you add the hammer/yes and gaprindashvili/yes labels

@bdunne bdunne merged commit e65f4d3 into ManageIQ:master Oct 19, 2018
@eclarizio
Copy link
Member Author

@miq-bot add_label gaprindashvili/yes, hammer/yes

simaishi pushed a commit that referenced this pull request Oct 22, 2018
Add product setting default for allowing API service ordering

(cherry picked from commit e65f4d3)

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

Hammer backport details:

$ git log -1
commit b310e6f18f068213165a6b568c91c451720e3600
Author: Brandon Dunne <brandondunne@hotmail.com>
Date:   Fri Oct 19 15:08:17 2018 -0400

    Merge pull request #18029 from eclarizio/BZ1632416-Addendum
    
    Add product setting default for allowing API service ordering
    
    (cherry picked from commit e65f4d354f2f15a07f2417692b6c5ce0f5182916)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1632416

simaishi pushed a commit that referenced this pull request Nov 5, 2018
Add product setting default for allowing API service ordering

(cherry picked from commit e65f4d3)

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

simaishi commented Nov 5, 2018

Gaprindashvili backport details:

$ git log -1
commit d5feeaaa2a80148ba582f2f27b6edf5d7814ca1c
Author: Brandon Dunne <brandondunne@hotmail.com>
Date:   Fri Oct 19 15:08:17 2018 -0400

    Merge pull request #18029 from eclarizio/BZ1632416-Addendum
    
    Add product setting default for allowing API service ordering
    
    (cherry picked from commit e65f4d354f2f15a07f2417692b6c5ce0f5182916)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1646435

nizaminabeel pushed a commit to nizaminabeel/manageiq that referenced this pull request Dec 11, 2018
Add product setting default for allowing API service ordering

(cherry picked from commit e65f4d3)

https://bugzilla.redhat.com/show_bug.cgi?id=1646435
@agrare agrare added this to the Sprint 97 Ending Oct 22, 2018 milestone Apr 4, 2019
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

6 participants