Last Comment Bug 436967 - Download Day map data should be made available as JSON with callback
: Download Day map data should be made available as JSON with callback
Status: RESOLVED FIXED
:
Product: Websites Graveyard
Classification: Graveyard
Component: spreadfirefox.com (show other bugs)
: unspecified
: All All
: -- enhancement (vote)
: ---
Assigned To: Les Orchard [:lorchard]
:
Mentors:
http://dev.spreadfirefox.com/en-US/wo...
Depends on: 437089
Blocks: 437151
  Show dependency treegraph
 
Reported: 2008-06-02 18:05 PDT by Les Orchard [:lorchard]
Modified: 2011-08-25 09:49 PDT (History)
3 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Patch to make Download Day map data available as JSON (2.17 KB, patch)
2008-06-02 18:05 PDT, Les Orchard [:lorchard]
ryan.doherty: review+
Details | Diff | Splinter Review
Revision to previous patch, now using json_encode() (1.99 KB, patch)
2008-06-03 10:38 PDT, Les Orchard [:lorchard]
ryan.doherty: review+
Details | Diff | Splinter Review

Description Les Orchard [:lorchard] 2008-06-02 18:05:01 PDT
Created attachment 323477 [details] [diff] [review]
Patch to make Download Day map data available as JSON

To facilitate third-party widgets and apps, it would be nice if the data powering the embedded map were made available in JSON format for cross-domain loading.

Attached is a patch that implements this.
Comment 1 Ryan Doherty (:rdoherty) 2008-06-03 10:15:12 PDT
Comment on attachment 323477 [details] [diff] [review]
Patch to make Download Day map data available as JSON

Looks good to me.
Comment 2 Les Orchard [:lorchard] 2008-06-03 10:38:59 PDT
Created attachment 323580 [details] [diff] [review]
Revision to previous patch, now using json_encode()

As it turns out, I don't have SVN checkin permission to this project...  So, here's another patch, using json_encode() without hacks.  I'll see if I can get access to check this in.
Comment 3 Ryan Doherty (:rdoherty) 2008-06-03 13:15:30 PDT
Comment on attachment 323580 [details] [diff] [review]
Revision to previous patch, now using json_encode()

Looks good.
Comment 4 Les Orchard [:lorchard] 2008-06-12 18:11:26 PDT
Finally checked this in as r15436

Note You need to log in before you can comment on or make changes to this bug.