search

agiliq / django-graphos

Django charting made *really* easy.
http://agiliq.com/demo/graphos
BSD 2-Clause "Simplified" License
442 stars 98 forks source link

Update utils.py #149

Open hcars opened 4 years ago

hcars commented 4 years ago

Fixed the problem for the newer versions of django with the six import.

eriktelepovsky commented 3 years ago

+1

hcars commented 3 years ago

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?

I just added it to the requirements.txt file.

utking commented 3 years ago

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.

hcars commented 3 years ago

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.

eriktelepovsky commented 2 years ago

I presume this library is dead...

woodmicha commented 2 years ago

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 https://github.com/agiliq/django-graphos/pull/149#issuecomment-1008002224, 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: @.***>