MMM-OpenWeatherForecast icon indicating copy to clipboard operation
MMM-OpenWeatherForecast copied to clipboard

fix: use openweather api 3.0

Open xmanwms95 opened this issue 1 year ago • 10 comments

openweather api 2.5 will be deprecated in June 2024 https://openweathermap.org/one-call-transfer

xmanwms95 avatar Apr 16 '24 02:04 xmanwms95

@xmanwms95 Since @jclarke0000 has not responded to issues for a long time and generally seems to have no activity on his GitHub account, it might be a good idea to continue maintaining the module in a fork. Would you be willing to do that?

KristjanESPERANTO avatar Apr 19 '24 20:04 KristjanESPERANTO

Please do that, xman. If J. Clarke comes back, he can always pick it back up. In the meantime, we've got fixes.

BKeyport avatar Apr 29 '24 22:04 BKeyport

Sh... Saw this pull request to late and made a fork which allows to configure the URL as a config option.

Tom-Hirschberger avatar May 20 '24 13:05 Tom-Hirschberger

Sh... Saw this pull request to late and made a fork which allows to configure the URL as a config option.

Even better :+1: Can we switch to your fork?

KristjanESPERANTO avatar May 20 '24 18:05 KristjanESPERANTO

Sure if you like. But i can only guarantee for basic support. I use the module by my self and will try to keep it running. But I am very busy with my new born twins at the moment.

Make sure to set the new base url in the config as the fork still uses the 2.5 API in default. The instructions are part of the updated Readme.

Tom-Hirschberger avatar May 20 '24 18:05 Tom-Hirschberger

But I am very busy with my new born twins at the moment.

Congrats! That is definitely a challenge. Best wishes Alex

lxne avatar May 20 '24 18:05 lxne

Basic support would be awesome. But even if you can not provide support, a working fork would already be a good step ahead.

Would be nice if you would accept the change of the clone command: https://github.com/Tom-Hirschberger/MMM-OpenWeatherForecast/pull/1

P.S.: Congratulations on the twins! 👍😀

KristjanESPERANTO avatar May 20 '24 18:05 KristjanESPERANTO

Thanks and accepted

Tom-Hirschberger avatar May 20 '24 18:05 Tom-Hirschberger

Maybe this should be merged now https://github.com/jclarke0000/MMM-OpenWeatherForecast/issues/36

goldyfruit avatar Oct 06 '24 14:10 goldyfruit

Hi in my fork the problem is solved since months

Tom-Hirschberger avatar Oct 06 '24 15:10 Tom-Hirschberger