censusreporter / census-api

The home for the API that powers the Census Reporter project.
MIT License
166 stars 50 forks source link

Control order of release to try #83

Closed JoeGermuska closed 4 years ago

JoeGermuska commented 4 years ago

This small patch uses a list instead of a set for releases to try in show and download, to see if that's why we're seeing different behavior on local and production. It also adds slightly clearer logging and ups the level to info.

Pausing on push just because I have to do something else, in case anything goes wrong. @iandees if you want to give this a try and or push it, please do.

This is to address the question from Slack:

I get different releases for this locally and remotely: https://api.censusreporter.org/1.0/data/show/latest?table_ids=B01003&geo_ids=16000US1714000 http://localhost:5000/1.0/data/show/latest?table_ids=B01003&geo_ids=16000US1714000 can’t figure out why that would be

iandees commented 4 years ago

This makes sense. Merging and deploying.