c5te1n / node-red-contrib-dwd-local-weather

Node Red node to retrieve local weather forecast from DWD (Germany)
Apache License 2.0
13 stars 11 forks source link

update error #8

Closed w0nk1 closed 4 years ago

w0nk1 commented 5 years ago

I just tried out your dwd skill and run into this error when monitoring the output over the debug node:

TypeError: updateWeatherForecastIfOutdated(...).then(...).catch(...).finally is not a function

Is this a known issue?

c5te1n commented 5 years ago

I have not seen this before, but from having a quick look, this might happen if you are using a nodejs version <10.0.0. Are you?

w0nk1 commented 5 years ago

Indeed. It is v8.12.0. I used it with the ioBroker Version of NodeRed. Is it possible to make it backward compatible?

Bundeshorst commented 5 years ago

same problem here. Any chance to run it with nodejs 8.12.0?

c5te1n commented 5 years ago

The latest version of ioBroker seems to come bundled with nodeJS 10. Is it an option to upgrade to this instead? See: https://www.iobroker.net/#de/download

renne commented 5 years ago

Same situation here. I don't know which version of node.js is used by ct-Smarthome, but node-red-contrib-dwd-local-weather publishes once on start-up and shows Requesting latest MOSMIX data file forever. Triggering an inject node manually doesn't help.

c5te1n commented 4 years ago

Closing this as the Docker container ct-Smart-Home is based on nodered/node-red:latest-minimal, which is on node 10 right now.