Skip to content

Made haystack-2.0.0beta compatible #8

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

pannal
Copy link

@pannal pannal commented Feb 24, 2012

This makes django-cms-search haystack-2.0.0beta compatible in a pretty clean way.

@BertrandBordage
Copy link

I have not looked at the code, but this commit works perfectly on my haystack-2.0.0beta installation since weeks.
Why not pulling this?

@beniwohli
Copy link
Owner

Mainly because haystack 2.0 is not out yet. AFAICT, it's not possible to support both haystack 1.2 and 2.0-beta at the same time.

Once haystack 2.0 is out, I'll look into supporting it.

@BertrandBordage
Copy link

Ok, twas a silly question :)
I thought you could be interested in starting a new branch to prepare django-cms-search to be compatible with haystack 2.0 when it would be out.

@JoeJasinski
Copy link

Hi, I'd be interested in using a this with support for haystack 2.0. Would it be possible to create this 2.0 development branch, perhaps with BertrandBordage's codebase as a starting point, so we could consolidate any work towards haystack 2.0 support in one location? Or should I fork off of BertrandBordage's branch and he can merge at the appropriate time?

@foobacca
Copy link

Any news on this? Haystack is now at 2.1.0 and 2.0 was released in May 2013 - six months ago.

@beniwohli
Copy link
Owner

@foobacca the bad news is that I don't plan to update django-cms-search to haystack 2, it's just way to messy. The good news is that there's a new module called aldryn-search over at https://github.com/aldryn/aldryn-search which works with haystack 2. I didn't really announce it yet because "aldryn" as a whole (https://www.django-cms.org/en/aldryn/) isn't officially announced yet. But we're using aldryn-search in production and it works quite well.

@grigoryk
Copy link

@piquadrat you might still want to put a note about aldryn-search into the README of the project. It might save people some time digging around 👍

@johnraz
Copy link

johnraz commented Jan 21, 2014

+1 to the readme addition

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.

7 participants