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

Ensure ServiceTemplate ordering passes through the submit_workflow flag #504

Merged

Conversation

eclarizio
Copy link
Member

@eclarizio eclarizio commented Oct 30, 2018

When I made #476, #485 got merged before it, and I overlooked an issue where the submit_workflow flag was not being set correctly, causing issues when submitting requests directly through the API instead of the UI. When the backend attempts to validate the dialog, the correct values that were passed through the request are not loaded without the submit_workflow flag set.

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

@miq-bot add_label bug, hammer/yes, gaprindashvili/yes
@miq-bot add_reviewer @d-m-u
@miq-bot assign @bdunne

@miq-bot
Copy link
Member

miq-bot commented Oct 30, 2018

Checked commit eclarizio@f5ff338 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. 🍰

@bdunne bdunne merged commit bed1032 into ManageIQ:master Oct 31, 2018
@bdunne bdunne added this to the Sprint 98 Ending Nov 5, 2018 milestone Oct 31, 2018
simaishi pushed a commit that referenced this pull request Nov 1, 2018
…_addendum

Ensure ServiceTemplate ordering passes through the submit_workflow flag

(cherry picked from commit bed1032)

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

simaishi commented Nov 1, 2018

Hammer backport details:

$ git log -1
commit 8475b1b2b099e9905231f7b0e39de84bfa752305
Author: Brandon Dunne <brandondunne@hotmail.com>
Date:   Wed Oct 31 15:00:05 2018 -0400

    Merge pull request #504 from eclarizio/dialog_ordering_security_issue_addendum
    
    Ensure ServiceTemplate ordering passes through the submit_workflow flag
    
    (cherry picked from commit bed1032d1e1fe54926e6717f116ff89cf5b55414)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1632416

simaishi pushed a commit that referenced this pull request Nov 5, 2018
…_addendum

Ensure ServiceTemplate ordering passes through the submit_workflow flag

(cherry picked from commit bed1032)

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 a2572b3838432e92a573a348ac34da357edc3569
Author: Brandon Dunne <brandondunne@hotmail.com>
Date:   Wed Oct 31 15:00:05 2018 -0400

    Merge pull request #504 from eclarizio/dialog_ordering_security_issue_addendum
    
    Ensure ServiceTemplate ordering passes through the submit_workflow flag
    
    (cherry picked from commit bed1032d1e1fe54926e6717f116ff89cf5b55414)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1646435

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