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

Raise error when creating displayed catalog item without dialog #2057

Merged
merged 1 commit into from Mar 17, 2015
Merged

Raise error when creating displayed catalog item without dialog #2057

merged 1 commit into from Mar 17, 2015

Conversation

skateman
Copy link
Member

@skateman skateman commented Mar 9, 2015

@dclarizio
Copy link

@skateman Add now raises error without a dialog chosen, but also raises the error if I choose one.

@skateman
Copy link
Member Author

@dclarizio now it should be okay

@dclarizio
Copy link

@skateman Not sure if I'm doing something wrong, but I pulled updates from your branch and get the same results:
add catitem error

@skateman
Copy link
Member Author

@dclarizio now it should be really okay :)

@miq-bot
Copy link
Member

miq-bot commented Mar 13, 2015

Checked commit skateman@fbc6823 with rubocop 0.27.1
1 file checked, 0 offenses detected
Everything looks good. 🍪

@dclarizio
Copy link

Verified fix.

dclarizio pushed a commit that referenced this pull request Mar 17, 2015
Raise error when creating displayed catalog item without dialog
@dclarizio dclarizio merged commit 9ce7496 into ManageIQ:master Mar 17, 2015
@dclarizio dclarizio added this to the Sprint 21 Ending Mar 30, 2015 milestone Mar 17, 2015
@skateman skateman deleted the service-catalog-dialog-fix-fix branch March 18, 2015 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants