Default Weather module still not displaying

[Jarhead96097]

@sdetweil no luck. Still receiving the same error on line 159, and no display on the MM.

[sdetweil]

@Jarhead96097

this is on pi, or windows, or ??

ok, do this

cd ~/MagicMirror
rm -rf node_modules
npm install 
#added
git checkout modules/default/weather/weather.js
git status

then try again

[Blackeye]

@Jarhead96097 Could be the comma after “true” on the last line of config … not necessary, and causes an error when I run a :check

[sdetweil]

@Jarhead96097

I added a command above to restore the original weather.js from the git repo

[Jarhead96097]

@sdetweil I added “npm audit fix” as there was 1 high severity vulnerability found. It didn’t solve the vulnerability (didn’t try “npm audit fix --force”), and I’m getting the same error.

[Jarhead96097]

@sdetweil Oh, okay. Let me try that updated string of commands…

[sdetweil]

@Jarhead96097 please never do audit… causes more problems than it fixes

[mumblebaj]

@Jarhead96097 Rule of thumb is to never run npm audit fix.

[sdetweil]

@mumblebaj and we have found npm install parms to turn off the warning coming in next release, and already in my scripts.

[mumblebaj]

@sdetweil That’s great