Skip to content

Fixes for Django 3#31

Open
darkman66 wants to merge 1 commit into
ouhouhsami:masterfrom
darkman66:master
Open

Fixes for Django 3#31
darkman66 wants to merge 1 commit into
ouhouhsami:masterfrom
darkman66:master

Conversation

@darkman66

Copy link
Copy Markdown

No description provided.

@glassresistor

Copy link
Copy Markdown

This looks like the fix that i need

@Julio0250 Julio0250 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yesterday I made the same changes because reverse is not declared when evaluate the version of django if u use django 3. This package also works en python 3.4+

@ericswpark

Copy link
Copy Markdown

@ouhouhsami please merge this change!


if DJANGO_VERSION[0] == 2:

if DJANGO_VERSION[0] in (2, 3):

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To support also Django 4+ just use:

if DJANGO_VERSION[0] == 1:
    from django.core.urlresolvers import reverse
else:
    from django.urls import reverse

Comment thread README.rst
Requirements :
* Django 1.7.4 or above.
* Django 1.7.4 or above. (also 3.x)
* Python 2.7 or 3.4

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python 2.7 or 3.4+ should be fine and Django 4 either.

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.

5 participants