Skip to content
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

support 1.7 and 1.6 request #4

Closed
wants to merge 33 commits into from
Closed

support 1.7 and 1.6 request #4

wants to merge 33 commits into from

Conversation

gpxlcj
Copy link

@gpxlcj gpxlcj commented Nov 17, 2016

Hello,

I think it will be better to continue supporting the version under 1.8. And I try to add the origin code into the /django_markdown/urls.py

Thanks for your reading.


from django.conf.urls import url
from django import VERSION as version

from .views import preview

if (version[0] <=1) and (version[1] < 8):
    from django.conf.urls import patterns
    urlpatterns = patterns('', url('preview/$', preview, name='django_markdown_preview'))
else:
    urlpatterns = [
        url('preview/$', preview, name='django_markdown_preview')
    ]

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 64.224% when pulling 9c552c0 on master into ef4888d on develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 64.224% when pulling 0bcf06b on master into ef4888d on develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 64.224% when pulling 6d10ade on master into ef4888d on develop.

@sv0
Copy link
Owner

sv0 commented Mar 18, 2017

If you want to use django-markdown with Django<1.8 you can install
django-markdown==0.8.2 instead of latest one.

@sv0 sv0 closed this Apr 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants