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

Menu Simplification: move Utilization from Optimize to Cloud Intel #5448

Merged
merged 1 commit into from
Apr 12, 2019

Conversation

skateman
Copy link
Member

@skateman skateman commented Apr 11, 2019

@miq-bot
Copy link
Member

miq-bot commented Apr 11, 2019

Checked commit skateman@c6e0d86 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@skateman
Copy link
Member Author

@miq-bot add_label enhancement, optimize, hammer/no, ux/review
@miq-bot add_reviewer @mzazrivec
@miq-bot add_reviewer @epwinchell

I am not sure about the order of the menu items, we might want to discuss it in the UI room + involve @terezanovotna and @Loicavenel.

@skateman skateman changed the title [WIP] Menu Simplification: move Utilization from Optimize to Cloud Intel Menu Simplification: move Utilization from Optimize to Cloud Intel Apr 11, 2019
@miq-bot miq-bot removed the wip label Apr 11, 2019
Copy link
Contributor

@epwinchell epwinchell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested. Works fine.

@mzazrivec
Copy link
Contributor

@terezanovotna Can you comment / approve please? Thanks.

@terezanovotna
Copy link

@miq-bot remove_label ux/review
@miq-bot add_label ux/approved

@mzazrivec mzazrivec self-assigned this Apr 12, 2019
@mzazrivec mzazrivec added this to the Sprint 109 Ending Apr 15, 2019 milestone Apr 12, 2019
@mzazrivec mzazrivec merged commit 35b4281 into ManageIQ:master Apr 12, 2019
@skateman skateman deleted the move-utilization branch April 12, 2019 14:33
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.

5 participants