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

Fixed some issues introduced with retirement remove resources option #697

Merged
merged 1 commit into from Mar 16, 2017

Conversation

h-kataria
Copy link
Contributor

@h-kataria h-kataria commented Mar 15, 2017

Also replaced .find calls with .find_by.

@gmcculloug please test for issues found during catalog item edit.

@lgalis is looking into fixing save button being enabled on initial load of edit form in a separate PR.

@gmcculloug
Copy link
Member

👍 This is working for me. Thanks

@h-kataria h-kataria removed the wip label Mar 16, 2017
@h-kataria h-kataria changed the title [WIP] Fixed some issues introduced with retirement remove resources option Fixed some issues introduced with retirement remove resources option Mar 16, 2017
@h-kataria h-kataria force-pushed the catalog_item_fixes branch 2 times, most recently from 3628ec9 to 3be3e54 Compare March 16, 2017 21:26
@dclarizio dclarizio self-assigned this Mar 16, 2017
- Also replaced .find calls with .find_by.
- Fixed broken spec tests with recent changes.

https://www.pivotaltracker.com/story/show/141238703
@miq-bot
Copy link
Member

miq-bot commented Mar 16, 2017

Checked commit h-kataria@25f8cda with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 12 offenses detected

app/controllers/catalog_controller.rb

@dclarizio dclarizio merged commit d4f095e into ManageIQ:master Mar 16, 2017
@dclarizio dclarizio added this to the Sprint 57 Ending Mar 27, 2017 milestone Mar 16, 2017
@h-kataria h-kataria deleted the catalog_item_fixes branch March 21, 2017 21:21
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

4 participants