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

[EUWE] Allow configuration managers providers and configuration scripts trees to display the advanced search box #13763

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Feb 3, 2017

Allow configuration managers providers and configuration scripts trees to display the advanced search box

Links [Optional]

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

Backport for ManageIQ/manageiq-ui-classic#295

Steps for Testing/QA

Steps to Reproduce:

  1. Add Tower provider
  2. Navigate to Compute -> Cloud -> Providers
  3. Navigate back to Config. Management

@miq-bot
Copy link
Member

miq-bot commented Feb 3, 2017

Checked commit lgalis@8dc2d4e with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
3 files checked, 0 offenses detected
Everything looks good. 🍪

@lgalis lgalis changed the title [WIP] [EUWE] Allow configuration managers providers and configuration scripts trees to display the advanced search box [EUWE] Allow configuration managers providers and configuration scripts trees to display the advanced search box Feb 3, 2017
@lgalis
Copy link
Contributor Author

lgalis commented Feb 3, 2017

@miq-bot remove_label wip

@dclarizio
Copy link

Change look good to me, same as upstream PR in ui-classic

@lgalis
Copy link
Contributor Author

lgalis commented Feb 3, 2017

@miq-bot add_label bug, blocker

@lgalis
Copy link
Contributor Author

lgalis commented Feb 3, 2017

@miq-bot assign @simaishi

@simaishi simaishi merged commit 68f5c52 into ManageIQ:euwe Feb 6, 2017
@simaishi simaishi added this to the Sprint 54 Ending Feb 13, 2017 milestone Feb 6, 2017
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.

4 participants