search

spencermountain / Freebase.js

inference and inspection on freebase data
107 stars 14 forks source link

Fixed place_data function to work with latest geosearch api #6

Closed jewelsjacobs closed 10 years ago

spencermountain commented 10 years ago

aww, hi julia. thank you for the kind words. i didn't know it was possible to 'within radius:...' on the search api! coooooool! i'm happy to work on this this week. This merge request contains a lot of stuff, and i can't believe i was foolish enough to commit the node_modules folder. let me clean things up, and figure out a way to add you as a committer, that way. no worries making the new mod! i'm not one of those people.... will let you know how it goes in the next few days. cheers

spencermountain commented 10 years ago

hey julia, i've made these geo api changes, along with a bunch of bugfixes, let me know if it's gonna do the trick? thanks for the heads-up

jewelsjacobs commented 10 years ago

Can't wait to check it out! I'll give it a whirl and let you know if I run into anything. Thanks for looking into this so quickly. Now if I could just get protractor to approve and add my pull request I won't have to hack any node mods!

jewelsjacobs commented 10 years ago

place_data works like a charm! thanks so much yet again. :)