-
Notifications
You must be signed in to change notification settings - Fork 98
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
Update utils.py #149
base: master
Are you sure you want to change the base?
Update utils.py #149
Conversation
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.
I agree with this fix, since six
is not supported by newer versions of django and without this fix the code doesn't run.
Do you also need to update the django-graphos/demo_project/requirements.txt
file to include six
?
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 needed for sure! Thanks for doing it.
+1 |
Added `six` to requirements.txt
I just added it to the requirements.txt file. |
Are there plans to merge this PR in the fairly near future? It allows the library to be working with the current Django versions, finally. |
@grchang2020 please approve the changes when you can. |
I presume this library is dead... |
We're still using it. Just made a copy and edit as needed.
…On Sat, Jan 8, 2022 at 6:07 AM Erik Telepovský ***@***.***> wrote:
I presume this library is dead...
—
Reply to this email directly, view it on GitHub
<#149 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE5JNM3TWP6TTT3BZZ7LJ4LUVBALDANCNFSM4LXHCGKQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you commented.Message ID:
***@***.***>
|
Fixed the problem for the newer versions of django with the six import.