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

Django 1.9 compatibility #19

Merged
merged 1 commit into from Nov 12, 2015
Merged

Django 1.9 compatibility #19

merged 1 commit into from Nov 12, 2015

Conversation

chris48s
Copy link
Contributor

@chris48s chris48s commented Nov 7, 2015

In Django 1.9b1, this module throws the error:
"django.core.exceptions.AppRegistryNotReady: Apps aren't loaded yet."
This is because it is no longer possible to run model imports in the root package (as specified in the Django 1.9 release notes: https://docs.djangoproject.com/en/1.9/releases/1.9/ ).
While the diff makes it look like a lot has changed in this commit, all I've actually done is move everything to models.py and leave init.py blank. I've tested this back to Django 1.5 and it does not break backwards compatibility.

Additionally, when Django 1.9 is released in December, version 1.7 will no longer receive security updates (as per Django's support policy: https://docs.djangoproject.com/en/1.8/internals/release-process/#supported-versions ). It would be helpful if there was a copy of this extension on pypi that is compatible with current supported versions of Django. Is there any plan to package up the current iteration (0.4?) and publish to pypi?
Download statistics on https://pypi.python.org/pypi/djorm-ext-pgtrgm/0.3 indicate that there is interest in the module and users are actively downloading the module from this channel.

As of Django 1.9 it is no longer possible to run model imports in the root package
jleivaizq added a commit that referenced this pull request Nov 12, 2015
Django 1.9 compatibility
@jleivaizq jleivaizq merged commit 6a5eefd into jleivaizq:master Nov 12, 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