RyanBalfanz / django-sendgrid

SendGrid for Django
http://pypi.python.org/pypi/django-sendgrid
97 stars 26 forks source link

Use built-in json instead of simplejson #66

Closed blag closed 10 years ago

blag commented 10 years ago

Update for Django 1.6

RyanBalfanz commented 10 years ago

Thanks, @blag. Did you happen to search the code for other uses of simplejson which should be replaced?

I'll merge this once any other uses are properly handled.

blag commented 10 years ago

Good catch. I removed the imports from views.py and message.py because they were not used.

These commits are untested because I'm not at my usual computer [editied through Github's web interface].

There is an import json, without a try/except wrapper, in header.py. I'm not sure what you want to do with that.

Let me know if you'd like me to do anything else.

shangxiao commented 10 years ago

This is absolutely necessary for use with 1.7 - any updates? :)

RyanBalfanz commented 10 years ago

I'm on vacation right now so won't be able to have a look for another day or two, unfortunately.— Sent from Mailbox

On Thu, Jun 5, 2014 at 4:36 AM, David Sanders notifications@github.com wrote:

This is absolutely necessary for use with 1.7 - any updates? :)

Reply to this email directly or view it on GitHub: https://github.com/RyanBalfanz/django-sendgrid/pull/66#issuecomment-45208798