search

koopjs / koop-provider-ckan

CKAN provider for Koop (experimental).
Other
4 stars 5 forks source link

Better 404 Handling #6

Closed chelm closed 9 years ago

chelm commented 9 years ago

Just adding a check for statusCode on responses from CKAN

chelm commented 9 years ago

Closes #5

jgravois commented 9 years ago

saweet. dmart and i will make sure to update the provider readme with new test urls too.

jgravois commented 9 years ago

@dmartine and i will test/merge this PR as well.

jgravois commented 9 years ago

looks good. koop now responds with

Unable to retrieve data from https://data.hdx.rwlabs.org/datastore/dump/45771200-34c9-4d0d-9a88-7c58512af19b

@dmartine will follow up with an additional PR once he tracks down the latest airport data.

chelm commented 9 years ago

@jgravois I'm revisiting this issue for #8 and wondering why the 404 on this dataset appeared in the first place. Can't seem to remember.