Code4HR / local-art-api

Public Art Installations in Norfolk, VA
https://localartapi-code4hr.rhcloud.com/
GNU Affero General Public License v3.0
4 stars 4 forks source link

Use live data instead of flat files. #23

Open ghost opened 9 years ago

ghost commented 9 years ago
wbprice commented 9 years ago

For the images route, what about:

GET /exhibits/{id}/images // returns an image (array of images) for exhibit {id}
PUT /exhibits/{id}/images // route to post an image to exhibit {id}

seems simpler to me.

ghost commented 9 years ago

Resource for Ember.js REST Adapter.

qwo commented 9 years ago

i think i felt blocked when we had to write it as a service that integrates with localwiki and hosts data also on OSM which made it feel bloated. Re: OHI is 4 different repos, hrt-bus spans 3, writing smaller services distinct services make sense.

To unblock, I like you guys are just going with rolling your own API. Im thinking if any of those ^ priorities arise again we should just write separate integrations to port data back and forth and keep the architecture modular.

anyway +1 feel free also to blow away the code especially with the express->hapi transition and put this on a branch

wbprice commented 9 years ago

Re: item 1; I had hoped to build a tool that would import the flat file automatically to OSM, but it appears that the climate within the OSM community is chilly towards importing a lot of data all at once (especially as a first contribution). We should look at having a mini mapathon to add all the nodes to OSM manually.

ghost commented 9 years ago

Absolutely. I could also probably do this during breaks at work, honestly.

jonahadkins commented 9 years ago

im on it -- :+1:

ghost commented 9 years ago

:smiley: Sweet man, thanks.

wbprice commented 9 years ago

Exhibits route is implemented here:

http://localartapi-code4hr.rhcloud.com/exhibits
http://localartapi-code4hr.rhcloud.com/exhibits?bbox=36.75,-76.44,36.98,-76.13
ghost commented 9 years ago

@wbprice, does the bounding box work here?

wbprice commented 9 years ago

Yes. If you don't send a bounding box to the /exhibits route it will assume bbox=36.75,-76.44,36.98,-76.13, but you specify a larger or smaller box as you need to.

ghost commented 9 years ago

Wow, awesome, man!

Also, I just went through and updated the descriptions and fullimages in each OSM node, then committed the OpenShift URL to our repo. Once we can correct some style things with images on the front page, we should push that to production.

qwo commented 9 years ago

i think this has been implemented

ghost commented 9 years ago

We still have 5/7 subtasks that we have not implemented, this is not closed yet.