Closed edanaher closed 4 years ago
@edanaher Thank you for working on this! I haven't been able to keep up with the discussion, but I'll review it this week
@lfhbento - In the interests of getting this merged, I went ahead and removed the stale-weather-color bits, so now this is just tweaking the open-weathermap API usage. I'll open another PR for the stale-weather-color, including a check for PBL_COLOR to only enable it on color pebbles.
@edanaher Thank you for this! Looks good to me :) I'll merge it now.
These changes make OpenWeathermap work. This also includes a bit to change the color for weather data if it's stale, since updating still doesn't reliably work for me.
If there's interest or a chance of merging this, I could clean it up a bit more, but after recent discussion on #24, I figure it's better to get this out there rather than sitting on it for another year.