-
Notifications
You must be signed in to change notification settings - Fork 18
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
cater for changed API in Django 1.11 #15
Conversation
as mentioned on https://docs.djangoproject.com/en/1.11/releases/1.11/#database-backend-api: "The name keyword argument is added to the DatabaseWrapper.create_cursor(self, name=None) method to allow usage of server-side cursors on backends that support it." (currently only postgresql) also adjusted tox.ini and .travis.yml to consider the currently supported versions
Fixes #16 |
2 similar comments
Hi there. First, thanks for the great software ! Nice to see there's already a port to Django 1.11. |
Hi @nschlemm , we (Polyconseil) have taken over the maintenance of django-cid (with the nice support of the original author). I have started to clean up pull requests and modernize the project. Your pull request had changes which should be reflected in the following commits:
I see that you also wanted to clean up Feel free to contribute again if you feel like it. We use django-cid and are committed to maintain it. |
as mentioned on https://docs.djangoproject.com/en/1.11/releases/1.11/#database-backend-api:
"The name keyword argument is added to the DatabaseWrapper.create_cursor(self, name=None) method to allow usage of server-side cursors on backends that support it."
(currently only postgresql)
also adjusted tox.ini and .travis.yml to consider the currently supported versions