ADI-Labs / density

wireless density API
13 stars 26 forks source link

Overrides the default JSON encoder to write a ISO 8601 string #124

Closed natebrennand closed 9 years ago

natebrennand commented 9 years ago

So this was actually a JSON marshaler problem the whole time. Now I'm much more familiar with pyscopg2 anyways....

closes #40 closes #84

screenshot 2015-03-03 14 17 35


This does break backwards compatibility but this needs to be changed so that it's valid.

schlosser commented 9 years ago

variable names then lgtm

natebrennand commented 9 years ago

fixed but waiting on @thebrianzeng since this trashes backwards compatibility for anyone using it

thebrianzeng commented 9 years ago

lgtm, this might break chris mulligan's app though but it should be a quick fix if it does. should we keep a change log?

natebrennand commented 9 years ago

+1 to a changelog. but we should probably do semantic versioning as well at that point.

natebrennand commented 9 years ago

I added a tag to the master branch to put it at 0.1.0. You guys as a group should decide exactly what 1.0.0 is defined as and make a milestone.

big things necessary from my perspective: