-
Notifications
You must be signed in to change notification settings - Fork 1
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
IDs for Indicators are not static #1
Comments
Not sure I follow. Can you explain what changed? |
As an example, previously, the average driving time to the next airport was accessible by calling |
Such changes were prevalent with next to all indicators I am using. |
Oh, ok! Thanks for sharing. I will put a note in the Readme. |
I likely would have overlooked this and would have ended up with wrong data, if the ID used so far ( Is there a way to fix the IDs to a certain variable? Or is |
This is not an issue with the package but their (undocumented) API. |
I see. Do you know if the values of |
I don't know. |
Okay, thank you though :) |
Obviously as the JSON API is undocumented, they are not bound to the IDs static. I got a rude awakening when this morning my reachability indicators suddenly changed to unemployment rates.
This is clearly not in your responsibility, yet I think it would be beneficial to add this information to the readme, just so people should increase their test coverage to cover this possible problem.
The text was updated successfully, but these errors were encountered: