Finish JSON API #126

Closed
tmcw opened this Issue Oct 11, 2012 · 5 comments

Comments

Projects
None yet
3 participants
@tmcw
Contributor

tmcw commented Oct 11, 2012

I/we want to finish the JSON API work initially prototyped by Jeff Warren.

Basically this consists of adding & testing the json format from Rails, implementing this in cgimap if needed, and scoping out supporting GeoJSON on some calls so that the API results are more easily usable.

@jfirebaugh

This comment has been minimized.

Show comment
Hide comment
@jfirebaugh

jfirebaugh Oct 11, 2012

Member

Do you have pointers to the prior work on this?

Member

jfirebaugh commented Oct 11, 2012

Do you have pointers to the prior work on this?

@tmcw

This comment has been minimized.

Show comment
Hide comment
@tmcw

tmcw Oct 11, 2012

Contributor

Jeff's branch is messy but here's a relevant commit.

Contributor

tmcw commented Oct 11, 2012

Jeff's branch is messy but here's a relevant commit.

@tomhughes

This comment has been minimized.

Show comment
Hide comment
@tomhughes

tomhughes Oct 11, 2012

Member

There isn't any meaningful work on this as far as I know, at least in the rails code.

I don't know what that code @tmcw refers to is - as far as I know it has never been shown to us before, and certainly wasn't discussed with us before it was written.

What anybody working on this does need to do is to work with @zerebubuth to make sure it is compatible with he has done/is planning in cgimap.

Member

tomhughes commented Oct 11, 2012

There isn't any meaningful work on this as far as I know, at least in the rails code.

I don't know what that code @tmcw refers to is - as far as I know it has never been shown to us before, and certainly wasn't discussed with us before it was written.

What anybody working on this does need to do is to work with @zerebubuth to make sure it is compatible with he has done/is planning in cgimap.

@tmcw

This comment has been minimized.

Show comment
Hide comment
@tmcw

tmcw Oct 11, 2012

Contributor

@tomhughes Read my comment; I link to the code. It's out of date, and needs tests.

Contributor

tmcw commented Oct 11, 2012

@tomhughes Read my comment; I link to the code. It's out of date, and needs tests.

@tomhughes

This comment has been minimized.

Show comment
Hide comment
@tomhughes

tomhughes Oct 11, 2012

Member

I tried, but the diff was so massive it killed my browser. That does not bode well as it suggests that a massive amount of work has been done without talking to anybody.

Member

tomhughes commented Oct 11, 2012

I tried, but the diff was so massive it killed my browser. That does not bode well as it suggests that a massive amount of work has been done without talking to anybody.

@tmcw tmcw closed this Jan 28, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment