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

Add delete functionality for generic object UI #12102

Closed
wants to merge 8 commits into from

Conversation

eclarizio
Copy link
Member

@eclarizio eclarizio commented Oct 21, 2016

This was built on top of #11815, so once that is merged, the first four commits here will disappear.

This enables the user to delete a selected generic object definitions via the configuration drop-down.

Before deletion:
screen shot 2016-10-21 at 9 03 14 am

After deletion:
screen shot 2016-10-21 at 9 01 15 am

https://www.pivotaltracker.com/story/show/129980083

@miq-bot assign @gmcculloug

@miq-bot
Copy link
Member

miq-bot commented Nov 2, 2016

<pr_mergeability_checker />This pull request is not mergeable. Please rebase and repush.

@miq-bot
Copy link
Member

miq-bot commented Nov 18, 2016

Checked commits eclarizio/manageiq@7083894~...2df237f with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
5 files checked, 0 offenses detected
Everything looks good. ⭐

@miq-bot
Copy link
Member

miq-bot commented Dec 22, 2016

<pr_mergeability_checker />This pull request is not mergeable. Please rebase and repush.

@chessbyte
Copy link
Member

Please re-create this PR in https://github.com/ManageIQ/manageiq-ui-classic

@chessbyte chessbyte closed this Jan 12, 2017
@eclarizio
Copy link
Member Author

@chessbyte Will do, currently waiting for ManageIQ/manageiq-ui-classic#72 to be merged as this is built on that and it should make fixing the conflicts easier.

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.

4 participants