ThingPulse / esp8266-weather-station

ESP8266 Weather Station library supporting OpenWeatherMap, Aeris and other sources
MIT License
1.06k stars 359 forks source link

Added "snow" to OpenWeatherMapOneCallDailyData. #192

Closed timrosskamp closed 3 years ago

timrosskamp commented 3 years ago

Make sure all boxes are checked (add x inside the brackets) when you submit your contribution, remove this sentence before doing so.

This PR is a small change to the One Call API. Currently only the "rain" key will be parsed in the daily forecast, although OWM also provides a "snow" key where available.

I have tested the change in a project of mine and it was working well. The change is small but I'm not entirely sure there are no side effects to my contribution.

marcelstoer commented 3 years ago

Thank you!