search

rpkilby / jsonfield

A reusable Django model field for storing ad-hoc JSON data
MIT License
1.16k stars 271 forks source link

Import error on Django master (3.0) due to django.utils.six #225

Closed therefromhere closed 4 years ago

therefromhere commented 5 years ago

django.utils.six has been removed on Django master, so this fails unless six is installed

try:
    from django.utils import six
except ImportError:
    import six
therefromhere commented 5 years ago

I think the short-term fix is to add six as a dependency, medium term is dropping py2 support so we don't need six.

Also the above lines need to be added to jsonfield/encoder.py

therefromhere commented 5 years ago

Ah this is less pressing than I thought - Django master is now for django 3.0 release, not 2.2.

therefromhere commented 4 years ago

Looks like this project is abandoned, we've migrated to https://github.com/rpkilby/jsonfield2 instead.

kstopa commented 4 years ago

I've already created a pull request solving this issue

dmkoch commented 4 years ago

I release version 2.1.0 that should address