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

Add haystack index #152

Merged
merged 27 commits into from Oct 21, 2015
Merged

Add haystack index #152

merged 27 commits into from Oct 21, 2015

Conversation

yakky
Copy link
Member

@yakky yakky commented Oct 17, 2015

No description provided.

@@ -0,0 +1,83 @@
# -*- coding: utf-8 -*-
from django.utils.encoding import force_text
from django.utils.text import force_unicode
Copy link
Member Author

Choose a reason for hiding this comment

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

Change with force_text

from .settings import get_setting

try:
from django.utils.encoding import force_text as force_unicode_or_text
Copy link
Member Author

Choose a reason for hiding this comment

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

this is not needed. Just use from django.utils.encoding import force_text it exists in every supported Django version

Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@yakky
Copy link
Member Author

yakky commented Oct 20, 2015

@fmarco I rebased on current develop with minimal changes to make it work. We need to check the desired behavior with multiple apphooks. Also: a test is failing apparently due to some leak between tests (test_blog_post_is_indexed_using_prepare are still in the index in test_blog_post_is_indexed_using_update_object). Apparently a index.clear() does not fix this

yakky added a commit that referenced this pull request Oct 21, 2015
@yakky yakky merged commit 7a839f8 into feature/appcnfig Oct 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants