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

[WIP] Add custom actions mixin to all models that require it #16369

Closed

Conversation

d-m-u
Copy link
Contributor

@d-m-u d-m-u commented Oct 31, 2017

This PR should be disregarded in favour of #16371

Adds the custom actions mixin to all models in https://docs.google.com/spreadsheets/d/11YAR-k40xCactWx8vKYU7udKtryGLMDuLQwK5d9kAnI/edit?ts=59f8c205#gid=0.

wip because needs UI components

Related to:
Host:
eclarizio@8ca5858
eclarizio/manageiq-ui-classic@f7d05b4
eclarizio/manageiq-api@e637616

Everything else:
ManageIQ/manageiq-api#163

@miq-bot miq-bot added the wip label Oct 31, 2017
@d-m-u
Copy link
Contributor Author

d-m-u commented Nov 1, 2017

@miq-bot assign @gmcculloug
@miq-bot add_label enhancement

@miq-bot
Copy link
Member

miq-bot commented Nov 1, 2017

Checked commits d-m-u/manageiq@fa2af84~...2b3d28e with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
49 files checked, 0 offenses detected
Everything looks fine. 👍

@d-m-u d-m-u closed this Nov 1, 2017
@d-m-u d-m-u deleted the adding_generic_custom_buttons_methods branch December 4, 2017 13:24
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.

3 participants