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

Fixes #5543/BZ1102322 - fix content search autocomplete. #4310

Merged
merged 1 commit into from Jun 19, 2014

Conversation

waldenraines
Copy link
Contributor

Autocomplete was broken for products, repositories, and
content views in different ways. This fixes each.

http://projects.theforeman.org/issues/5543
https://bugzilla.redhat.com/show_bug.cgi?id=1102322

Autocomplete was broken for products, repositories, and
content views in different ways.  This fixes each.
query {string name_query}
filter "and", [
{:terms => {:id => ids}},
{:terms => {:enabled => [true]}}
Copy link
Member

Choose a reason for hiding this comment

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

Is this covered by the readable calls above?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Which part, enabled? That no longer exists on on the repositories table.

@ehelms
Copy link
Member

ehelms commented Jun 19, 2014

ACK

waldenraines pushed a commit that referenced this pull request Jun 19, 2014
Fixes #5543/BZ1102322 - fix content search autocomplete.
@waldenraines waldenraines merged commit 698511e into Katello:master Jun 19, 2014
@waldenraines waldenraines deleted the 5543 branch June 19, 2014 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants