search

ThingPulse / esp8266-weather-station

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

OpenWeatherOneCall #215

Open Brown-onion opened 9 months ago

Brown-onion commented 9 months ago

The API request for 2.5 seems to not be supported by OpenWeatherOneCall anymore. I believe 3.0 is but haven't figured out how to change the URL in the library to test it. the library calls http://api.openweathermap.org:80/data/2.5/onecall And I believe it needs to call http://api.openweathermap.org:80/data/3.0/onecall This should be an easy fix, but I can't find the library file that contains it.

Cheers

marcelstoer commented 9 months ago

It's the OpenWeatherMap... files in the src folder at https://github.com/ThingPulse/esp8266-weather-station/tree/master/src.

Brown-onion commented 9 months ago

I thought it may have been in there but looked in OpenWeatherMapOneCall.h and OpenWeatherMapOneCall.cpp but couldn't find a http link to change. The serial print in the sketch reads '[HTTP] Requesting resource at http://api.openweathermap.org:80/data/2.5/onecall?appid........ Then my API key and other parameters in the program.

I'll keep looking anyway.

UPDATE: Line 42 of OpenWeatherMapCurrent.cpp the 2.5 needs to be changed to 3.0. But I don't believe that 2.5 works anymore so this library could use a version update to maintain it as current.

Feel free to close this as complete when updated.

CHEERS!