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.10 Compatibility #88
Conversation
- contidional import and return urls for Django >= 1.10, since patterns removed in 1.10
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please provide some tests to ensure the version comparisons are working properly?
Thanks for the PR! Looking forward to see the updates ;)
from django.conf import settings | ||
from django.conf.urls import patterns, url | ||
|
||
if get_django_version() >= '1.10.0': |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comparison might fail in some circumstances. Please see http://stackoverflow.com/questions/11887762/compare-version-strings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@seocam will be working on this.
🏃♂️
url(r'^(?P<pk>\d+)/$', cls.as_detail(), | ||
name=cls.build_url_name('detail', name_prefix))] | ||
|
||
if get_django_version() >= '1.10.0': |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.
django>=1.10 compatibility test initialized.
1 similar comment
Seems that Python 2.7 and 3.4 tests passed. see: http://stackoverflow.com/a/34648720/1766716 https://travis-ci.org/toastdriven/restless/jobs/173916340#L267 |
@guneysus maybe just a rebase will work. I saw that we are not testing 3.3 on the travis file. What do you think? |
Master should now be compatible with django 1.10 |
No description provided.