馃崚 weather/noaa: Adapt to server API changes / Improve the days shown in the forecast
Cherry-pick MR including two commits
weather/noaa: Improve the days shown in the forecast
Previously, a day would be filtered out from showing in the applet when it didn't have for instance the minimum temperature (which usually happens for the last day)
Let's filter out day forecasts only if they don't provide the weather condition, instead of min/max temperatures.
Also report the actual number of days after the filtering, instead of the total days in the report. Otherwise we could show less days than we claim on the tab.
(cherry picked from commit 79a96ffb)
weather/noaa: Adapt to server API changes
Recently, the weather applet has been failing to show forecast reports for the NOAA provider. This is a mix of the server rejecting some of the forecast requests and a small API change we weren't parsing correctly.
After some digging, this improves the chance of showing them right, or at least get a log when they don't.
-
Set a precision limit on the lat/lon parameters in the request to reduce the chance of the server rejecting it
-
When parsing the xml response, check for a
layout-key
that indicates daily forecast (24h), but not specifically 7-days, since now sometimes we also get an 8-days report -
In case of a server error issue, parse the returned message and log it. It can help future debugging but it's not really useful or actionable to the user.
(cherry picked from commit 35f624be)