Read the statement by Michael Teeuw here.
MMM-Netatmo does not load
-
@tommys great that you solved the issue! Would you mind contributing that back into the repository?
-
@CFenner normally I would say: Yes, absolutely! But… I’m not a javascript-developer and know very little about it and node.js.
This is the first time I dipped my toe in this world (I’ve been working for 30 years as a C++ developer :grinning_face_with_sweat: ) so I just tried things until I got it working. I have probably done something non-idiomatic or worse, so I think it is better that someone more knowledgable do a proper fix. But I’m happy to answer questions if more info than I already provided is needed.
Btw, a big thank you for your great Netatmo-module @CFenner! :folded_hands: -
Hello everybody, I have the same problem. Because I’m not an IT professional, I’m having trouble solving the problem with your instructions.
The second workaround in particular is difficult:
…I completely removed the usage of the sync-fetch library in netatmo.js in favor of the default async version . That is, simply remove the line: const fetch = require(‘sync-fetch’), make the authenticate, loadData and socketNotificationReceived functions async, handle the promise object returned by fetch appropriately as well as the call to json().- I can’t find the “const fetch” line in the netatmo.js file
- how can I make functions asynchronous
- the sentence “handle the promise object returned by fetch appropriately as well as the call to json().” I don’t really understand either
Is it possible to make the instructions a bit more detailed? Or what is the best way to ensure that the great module works for me again?
Thank you very much for your effort!
-
@tommys I have the same problem, but while I can follow instructions and edit files, your description is a bit to advanced for me.
Could you break it down, I would really like to get the module working again!
-
@cgillinger I merged a PR from @ottolote to replace the sync-fetch. Please try if that fixes the issue for you. It work for me now, but also did before that change using mm v2.26.
-
@CFenner And just to report back here as well: Now the module works for me again! Thnx!
-
@CFenner
Have you updated your module? Does it need to be reinstated or how to update it? -
I have reinstalled the entire module. And everything finally worked!) Thanks to the author of the module. I really like this module.
A huge only request to the author - add a pressure recalculation in mm Hg…!!!))) -
@Laz great that it works again. Sorry that is took so long. What about the RU translations, everything good?
Will try to find some time for the transformation. -
@CFenner Unfortunately, I now work remotely through VNC with a mirror, the resolution is not very high and all small details are not always clear. I will check the translation in April, as I will be in that city.
Thank you, I am waiting for a recount to other pressure units.!)
