-
Notifications
You must be signed in to change notification settings - Fork 898
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
Refactor Overview Utilization menu text #19062
Conversation
Checked commits GregP/manageiq@e3079f1~...159d256 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👎 This does not belong in core ManageIQ as it is part of a CloudForms-only plugin.
If shortcuts are not pluggable they need to be made pluggable first.
@@ -10,7 +10,7 @@ | |||
:rbac_feature_name: miq_report | |||
:startup: true | |||
- :name: miq_capacity_utilization | |||
:description: Cloud Intel / Utilization | |||
:description: Overview / Utilization |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be pulled out into a separate PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@h-kataria @bdunne @Fryguy Removed the commit with Red Hat Cloud menus, will mark up and comment the BZ as well. And renamed the PR to reflect only changes to Utilization menu.
@Fryguy @bdunne @h-kataria This is similar to another recent change, with Migration menus. Which are also a gem, and those were handled inside the gem code. I think something similar should be done here as well. This is a 2 part PR, first commit should still apply. Second one, for the Red Hat Cloud menus, I can delete for now until we figure out where it belongs. And maybe add a comment to the BZ itself once we reach a decision. |
I'm with @Fryguy , I don't really understand how the first commit is related to this PR |
@Fryguy @bdunne Its the other way around. Start at Login drop down lists all the miq shortcust, and this is how the BZ started. Red Hat Cloud was a later on "add on" to the BZ. Main reason I am saying that Red Hat Cloud commit should be deleted from the above, and first commit, dealing with refactoring of Utilization menu and main point of the BZ, should stay. I can change PR name to not reference Red Hat Cloud. |
159d256
to
e3079f1
Compare
Refactor Overview Utilization menu text (cherry picked from commit 499cb14) https://bugzilla.redhat.com/show_bug.cgi?id=1729136
Ivanchuk backport details:
|
Refactor Utilization menu to be listed under Overview.
https://bugzilla.redhat.com/show_bug.cgi?id=1729136
Screen shot of Utilization menu prior to code fix:
Screen shot of Utilization menu listed under Overview main menu post code fix: