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

Hide policy simulation button #9205

Merged

Conversation

GilbertCherrie
Copy link
Member

@GilbertCherrie GilbertCherrie commented Jun 13, 2024

Hide the policy simulation button on IM for the vms, templates, instances and images pages. The policy simulation page is currently broken. As we discuss how to progress with this page whether it is to fix the page, redesign and convert the page to react or delete the page all together we will hide the button that leads to this page.

Part of #8921

@miq-bot
Copy link
Member

miq-bot commented Jun 13, 2024

Checked commit GilbertCherrie@b4b03e1 with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
10 files checked, 1 offense detected

app/helpers/application_helper/toolbar/x_template_cloud_center.rb

@jeffbonson
Copy link
Contributor

Hey @GilbertCherrie , @Fryguy
do we need to take out these commented code blocks, or is it ok like how it is now?

@GilbertCherrie
Copy link
Member Author

Hey @GilbertCherrie , @Fryguy do we need to take out these commented code blocks, or is it ok like how it is now?

We can leave it as it is for now since we may decide to bring these buttons back later if the form is fixed

@jeffibm jeffibm merged commit 75e37b8 into ManageIQ:master Jun 14, 2024
15 checks passed
@GilbertCherrie GilbertCherrie deleted the hide_policy_simulation_button branch June 17, 2024 12:51
liu-samuel pushed a commit to liu-samuel/manageiq-ui-classic that referenced this pull request Jul 18, 2024
…lation_button

Hide policy simulation button
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants