-
Notifications
You must be signed in to change notification settings - Fork 5
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
Added support for Django 1.11 #38
Added support for Django 1.11 #38
Conversation
73a0867
to
111d14f
Compare
from django.contrib.messages import constants | ||
from django.core.urlresolvers import resolve, reverse, NoReverseMatch, Resolver404 | ||
from django.template.loader import get_template | ||
from django.template.base import token_kwargs, FilterExpression | ||
from django.template.loader_tags import do_extends, ExtendsNode | ||
from django.template.defaultfilters import slugify | ||
from django.utils.html import format_html | ||
from ast import literal_eval |
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.
literal_eval
isn't used anywhere
context_dict = context.flatten() | ||
return form_template.render(context_dict, context_dict.get('request')) | ||
else: | ||
return form_template.render(context) |
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.
.travis.yml
Outdated
@@ -28,5 +29,5 @@ deploy: | |||
distributions: "bdist_wheel sdist" | |||
on: | |||
all_branches: true | |||
condition: $DJANGO_VERSION = ">=1.10, < 1.11" | |||
condition: $DJANGO_VERSION = ">=1.10, < 2.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 is wrong - it needs to match one of the above entries.
e.g. here it should be condition: $DJANGO_VERSION = ">=1.11, < 2.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.
Fixed that
111d14f
to
e1b619c
Compare
No description provided.