-
Notifications
You must be signed in to change notification settings - Fork 557
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
yahoo api not working #144
Comments
I've been trying to get this to work as well, and discovered a notice on the Yahoo developer site (https://developer.yahoo.com/yql/) which reads : "Important EOL Notice: As of Thursday, Jan. 3, 2019, the YQL service at query.yahooapis.com will be retired. This will impact users of datatables.org as well as developers who creates features using this YQL service. So I'm assuming the existing code samples will no longer work with the old endpoints. |
This should be updated in the respective codelab (https://codelabs.developers.google.com/codelabs/your-first-pwapp/) as this is blocking the learning process. |
It still has not been updated I think, right?! |
What a pity this issue with https://query.yahooapis.com/ is preventing each stage (after the first one) of this codelab from working! As a previous comment said, "It's blocking the learning process." |
Here is a workaround: fake additional cities - we don't care about the actual api results as long as we can test the app.
app.updateForecastCard(additionalFakeForecasts[key]); This should unblock at least the city adding part of the tutorial |
Note that this will not 'save' the New York weather. So after refresh it will be replaced by the Austin weather. To fix this, i removed the [key] after additionalFakeForecasts and I also removed the --'2357536' : {-- in the Sample code block. I replaced the app.updateForecastCard in the else clause with: if (label === "New York, NY") { Of course this will add Austin in any case, even if you add another city. |
I've created an updated version of the app using OpenWeatherMaps. https://github.com/arshameDEV6/node-pwapp-playground Not sure what would be the best way to help update the original broken demo. My version does force users to use my node server. |
Piggybacking on some of the work that @MarkDuijf and @davidair did on making a workaround, I created a tweak to their solution that allows for all cities to be displayed. First, create a file in
Then add fake forecasts to
Finally, you can update the
It will appear as if all of the cities have the same weather, which is kinda lame; But it will allow you to (hopefully) complete the exercises |
@rgegriff Adding the code above works to have data available because the Yahoo API doesn't work anymore. But adding the
|
I submitted a pull request that updates this code lab for the new Yahoo API. |
Hi! Thanks for the feedback. I've rewritten the codelab from scratch and believe I have fixed this issue. The code has been updated, I'm just waiting for the guide to go live. In the meantime, I've deployed the codelab at https://developers.google.com/web/fundamentals/codelabs/your-first-pwapp/. If you find any issues with the new codelab, please let us know! Thanks! |
https://query.yahooapis.com/v1/public/yql?format=json&q=select%20*%20from%20weather.forecast%20where%20woeid=2367105
This api fails
The text was updated successfully, but these errors were encountered: