search

klen / django_markdown

Django markdown support and wysiwig
GNU Lesser General Public License v3.0
391 stars 143 forks source link

Preview view uses obsolete request.REQUEST #57

Closed alasdairnicol closed 12 months ago

alasdairnicol commented 8 years ago

Arising from this [stack overflow question](http://stackoverflow.com/questions/34588310/django-markdown-failed-to-show-preview-in-django-1-9/34588507#34588507 stack overflow question):

request.REQUEST was deprecated in Django 1.7, and removed in Django 1.9. The preview view needs to be updated to work with Django 1.9.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29567679-preview-view-uses-obsolete-request-request?utm_campaign=plugin&utm_content=tracker%2F332251&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F332251&utm_medium=issues&utm_source=github).
noisytoken commented 8 years ago

I forked the project and it is now working with django 1.9

Since request.REQUEST was deprecated in Django 1.7 and removed from Django 1.9, the new code read data from request.POST instead.

noisytoken commented 8 years ago

I made some changes for django 1.9 please review it

fosskers commented 8 years ago

Please merge this.

Mandragorian commented 8 years ago

Yeah, this should definitely be merged. Along with other django 1.10 compatibility related pathces. Archlinux for example distribute the 1.10 django version already.

jlariza commented 8 years ago

Please, merge this to be able to use it with Django 1.9

alasdairnicol commented 12 months ago

closing issue because the repo hasn't been updated in years