search

foundertherapy / django-cryptographic-fields

A set of fields that wrap standard Django fields with encryption provided by the python cryptography library.
MIT License
29 stars 34 forks source link

don't use SubfieldBase as its removed in django 1.10 #19

Closed GhalebKhaled closed 8 years ago

GhalebKhaled commented 8 years ago

The main change here that django.db.models.SubfieldBase is now removed. As suggested in this class, they suggest to use from_db_value, and this is what I did here, a related question in stackoverflow:

http://stackoverflow.com/questions/35166085/how-to-deal-with-subfieldbase-has-been-deprecated-use-field-from-db-value-inst

The other thing that we lost as django.db.models.SubfieldBase is removed now is setting model field as class field. You can see that some of the test below had to field in a different way. Hopefully I fixed it in the right now. @danaspiegel @waseem-omar Please review and let me know for any feedback.

GhalebKhaled commented 8 years ago

(2) 53 - Upgrade Django to 1.10

glow-mdsol commented 8 years ago

Could you also unpin the django version in requirements.txt as well?

GhalebKhaled commented 8 years ago

@glow-mdsol Updated - thanks