-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enrichment failing #9
Comments
Github ID appears to be an integer from your API, but it is a string in the user struct |
Oddly, the user twitter ID is an integer, and the company twitter ID is a string. That doesn't make sense @maccman :-P |
@philipithomas nice catch. Agree those should def be consistent.
|
Oops - should def be using strings for both. |
Related to #9 - will report back on whether it closes it
For the future, the is the best way I've seen for turning JSON into Go structs: |
Might be good to set up some basic integration testing on this library because Go is tricky with JSON integrations. Basically, have it query API and assert no error. It would be best to run this daily or so (which TravisCI could do) |
|
Does |
Getting this error in production:
It's stopping this library from completing enrichment.
The text was updated successfully, but these errors were encountered: