fix: made compilable for ESP32 (#186, #190)

ThingPulse / esp8266-weather-station

ESP8266 Weather Station library supporting OpenWeatherMap, Aeris and other sources

MIT License

1.06k stars 362 forks source link

Closed vlastahajek closed 3 years ago

vlastahajek commented 3 years ago

Closes #186, closes #190

[X] This PR is compliant with the other contributing guidelines as well (if not, please describe why).
[X] I have thoroughly tested my contribution.
[X] Should this code require changes to documentation I will contribute those to https://github.com/ThingPulse/docs.

Fixed compilation problems for ESP32.

marcelstoer commented 3 years ago

vlastahajek commented 2 years ago

When do you plan to merge ESP32 support, please?

marcelstoer commented 2 years ago

Oh, good point, thanks. I just rebased the ESP32 branch. Still haven't tested on ESP32 hardware but I assume you did?

vlastahajek commented 2 years ago

Yes, I'm using forecast and current weather API and it works in general.