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 1.8 incompatibility #1167 #1184

Closed
wants to merge 4 commits into from
Closed

Conversation

phingage
Copy link

@phingage phingage commented May 4, 2015

fix 1.8 incompatibility with deleted db.close_connection() internal API. see #1167

fix 1.8 incompatibility with deleted db.close_connection() internal API
@acdha
Copy link
Contributor

acdha commented May 4, 2015

This can't be merged in its current state because it's not compatible with Django 1.7 and earlier.

@acdha
Copy link
Contributor

acdha commented May 4, 2015

I'd like to drop support for older versions of Django in the not terribly distant future – maybe just toss in a deprecation warning when we have to fall back to close_connection and we can drop that in a future major release?

Fix 1.8 incompatibility with deleted db.close_connection() internal API,
added backward compatibility with version < 1.8
@phingage
Copy link
Author

phingage commented May 4, 2015

@acdha 71d6125 should be compatible with older version of django.

Fix 1.8 incompatibility with deleted db.close_connection() internal API,
added backward compatibility with version < 1.8 (Django warning
compliant)
@phingage
Copy link
Author

phingage commented May 4, 2015

@acdha Should be ok now (405bbf8)!

@acdha acdha self-assigned this May 29, 2015
@acdha acdha added this to the v2.4.0 milestone Jun 3, 2015
@acdha
Copy link
Contributor

acdha commented Jun 3, 2015

The Django 1.8 branch has merged as of bd8e512

@acdha acdha closed this Jun 3, 2015
@acdha
Copy link
Contributor

acdha commented Jun 5, 2015

There's a release candidate on PyPI now:

pip install django-haystack==2.4.0rc1

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.

None yet

2 participants