-
Notifications
You must be signed in to change notification settings - Fork 46
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
eurostat API broken #251
Comments
Thank you for the fix! I just want to report a bug in
|
Many thanks. We aim to look at this as soon as situation allows. Meanwhile, PR:s welcome, too, as always. |
Thank you! I overlooked that data downloaded through JSON API calls get processed by the same |
I see new version on cran |
Thanks for pointing out. We are having a look asap. |
Thanks, I was skimming through the issues and noticed unfortunately only now that there was the issue of date handling here. It is now fixed in my PR but unfortunately not in the CRAN version. I think we will make a new CRAN release when the old bulk download facilities are deprecated so the issue will be permanently fixed there, but in the mean time I recommend using the dev version from GitHub / r-universe. |
Thanks for the quick response @pitkant! Cold you fix it for quarters as well?
|
I think it should already work as intended for quarters as well. Here's my printout (eurostat 3.8.3 from https://github.com/rOpenGov/eurostat/tree/pitkant-4 branch):
|
I had to restart my session and now the date is ok. Thanks again! |
Permanent fix to the described issue is now in a stable release of eurostat package version 4.0.0. Closing this issue with the CRAN release of package version 4.0.0 |
It seems that Eurostat has turned off previous API?
But bulk download still works:
The text was updated successfully, but these errors were encountered: