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

Added "Remove Resources" on Catalog Item summary Retirement tab #764

Merged
merged 2 commits into from
Mar 22, 2017

Conversation

h-kataria
Copy link
Contributor

@gmcculloug @tinaafitz please review/test

before
before

after
after

@h-kataria
Copy link
Contributor Author

will need changes in #762 to be able to test this one.

@h-kataria h-kataria force-pushed the show_remove_resources_value branch from 0758023 to 5daf0dc Compare March 22, 2017 14:01
@chessbyte
Copy link
Member

@h-kataria #762 is merged

@h-kataria
Copy link
Contributor Author

@tinaafitz @gmcculloug please review/test.

@tinaafitz
Copy link
Member

@h-kataria I tested your changes and it looks good.

@miq-bot
Copy link
Member

miq-bot commented Mar 22, 2017

Checked commits h-kataria/manageiq-ui-classic@5daf0dc~...d2d1092 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. ⭐

@mzazrivec mzazrivec added this to the Sprint 57 Ending Mar 27, 2017 milestone Mar 22, 2017
@mzazrivec mzazrivec merged commit 49b560e into ManageIQ:master Mar 22, 2017
@h-kataria h-kataria deleted the show_remove_resources_value branch April 4, 2017 18:41
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.

6 participants