search

nasa-jpl-memex / GeoParser

Extract and Visualize location from any file
Apache License 2.0
52 stars 23 forks source link

Fixed the cross origin problem #61

Closed antrromet closed 8 years ago

antrromet commented 8 years ago

Hey @smadha So I looked online and its a common problem. I used jquery ajax call now instead of XML request. Its working fine now.

antrromet commented 8 years ago

@smadha In this though, I've added a failsafe. So if theres an error by any chance, it will show the default solr link.

antrromet commented 8 years ago

@smadha Ahh my bad! I didn't mean to push that in the PR. That was just for the demo for the assignment that I was working on!

smadha commented 8 years ago

Hey @antrromet as you see I already did the failsafe part so we don't need that. But I see you have also handled cross domain error by using jsonp. Can you please ensure everything works fine after I merge this PR to master. I see you merged latest changes in your PR but still want to double check.

antrromet commented 8 years ago

Hey @smadha, I just checked and I think its fine now. I also removed the search code for now. I will add a list of dropdown and then submit a PR for that later on.

chrismattmann commented 8 years ago

Guys I want @antrromet search code to be added have seen it and it's great cam we integrate it