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

Fix title in Config Settings page to match the accordion #3819

Merged

Conversation

hstastna
Copy link

@hstastna hstastna commented Apr 20, 2018

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1512443


Fix the title in the Configuration > Settings page when selecting ManageIQ Region (or CFME Region) in accordion, to match the name in the accordion.

Before:
region_before1

Editing the region (after clicking on Region10[10] in the list, under the title):
region_before2

After:
region_after1

Editing the region:
region_after2


Why I fixed this the way I fixed it:
See https://github.com/hstastna/manageiq-ui-classic/blob/master/app/presenters/tree_builder_ops_settings.rb#L18, especially the product.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1512443

Fix the title in the Configuration > Settings page when selecting
Region0 in accordion, to match the name in the accordion.
@hstastna
Copy link
Author

@miq-bot add_label bug, gaprindashvili/yes

@miq-bot
Copy link
Member

miq-bot commented Apr 20, 2018

Checked commit hstastna@47939e7 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. ⭐

@mzazrivec mzazrivec self-assigned this Apr 23, 2018
@mzazrivec mzazrivec added this to the Sprint 84 Ending Apr 23, 2018 milestone Apr 23, 2018
@mzazrivec mzazrivec merged commit 074d2e9 into ManageIQ:master Apr 23, 2018
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