-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Remove some deprecation warnings #4998
Conversation
e51bff9
to
81fa103
Compare
please review @divio/django-cms-core (and @timgraham ) |
request, "admin/cms/page/plugin/delete_confirmation.html", context, current_app=self.admin_site.name | ||
) | ||
else: | ||
return TemplateResponse( |
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.
Depending on the outcome of the flake8 max line length discusison, this is as example of where I'd use a single longer line (103 here).
d1dbec8
to
14b4697
Compare
@divio/django-cms-core @timgraham I'm going to merge this tomorrow , if there are no more objections |
The latest update will fail until a workaround for https://code.djangoproject.com/ticket/26041 is merged into classytags |
8d11f36
to
8089279
Compare
8089279
to
aad4895
Compare
Failing test here due to infamous related_name issue, I will merge this as soon as test suite completes |
More Django 1.9 work
Code should now be 100% deprecation warnings free
Tests still emits some (hunting down the remaining ones)