search

k-donn / plasmoid-wunderground

A Plasma 5/6 widget for showing data from Wunderground PWS
https://store.kde.org/p/2135799
GNU General Public License v2.0
35 stars 12 forks source link

High / Low not displaying - for Imperial #20

Closed rancod99 closed 2 years ago

rancod99 commented 3 years ago

High / Low not displaying - for Imperial units ( gets stuck on Loading..). Works just fine with Celsius.

Thank you,

k-donn commented 3 years ago

Ok, thanks for bringing up the issue.

To clarify, does the entire widget show the Loading ... screen or does just the top of the widget with the High/Low data show Loading; also, does the forecast page show up when the error occurs?

Screen shots would be helpful! Thanks!

rancod99 commented 3 years ago

image

rancod99 commented 3 years ago

image

k-donn commented 3 years ago

Thanks!

Going through the logic in: https://github.com/k-donn/plasmoid-wunderground/blob/b25cbb31ab41a07524b86b3ad91b1023e26f12fa/plasmoid/contents/code/pws-api.js#L112 If there were an error with the request, then an error message should show up on the entire widget. There is no central error message, therefore, there is not an error with the request. Because the High/Low part of the widget shows loading..., the variable showForecast must be false. Therefore, there is some processing error between lines 142-200.

Because this only occurs for imperial units, I think the JSON from Wunderground might be different for some reason.

Sorry, but I have some more questions.

Does this happen for all stations you choose or just a certain one?

Also, could you please show me your plasmashell log? Make sure Write to console.log is checked under the Debug settings in the widget. Then journalctl /usr/bin/plasmashell -f and change the units to imperial.

Thanks again for bringing up this issue.

k-donn commented 3 years ago

This is a normal log for choosing a station,