CANNOT ADD MODULES

lavolp3

and your indentation is messed up making it more difficult to find errors.
You may have too much curled brackets.

Mykle1

https://forum.magicmirror.builders/topic/4231/how-to-add-modules-for-absolute-beginners

yawns

I prefer jshint.com
In this case you missed a single comma in line userlon, so it should be:

		userlat: "47.302604", //MUST HAVE BOTH
	        userlon: "-122.33621",  //MUST HAVE BOTH
		zip: "98023"

lavolp3

@yawns said in CANNOT ADD MODULES:

I prefer jshint.com

This really is way better! Great! Doesn’t complain about tabs for indentation which is annoying with the other tool.

anhanyoung

Thank you guys! I will try using those sites. Didn’t know they existed. Oh man. Would that be a help. Another question, for d ample, the compliments and weather module, when I change it in the config, do I have to change it in the module folder as well? Adding more compliments and such?

lavolp3

@anhanyoung Every module is designed in a way that you don’t have to mess with the module files (MMM-*.js, node_helper.js) themselves.
Otherwise, if you’re not common with git, you will run into problems as soon as you want/need to update the module.
So always try to use config/config.js, css/customs.css or external files for tpurposes like these. In most caeses it’s described in the Readme of the config or here in the forum.

lavolp3

@anhanyoung said in CANNOT ADD MODULES:

when I change it in the config, do I have to change it in the module folder as well? Adding more compliments and such?

No, you only need to change it in the config.js, or in the external json. The module file contains default values that will be overwritten by the values in the config.js or external files (e.g. for compliments)

anhanyoung

@mykle1

Thank you. I have been using your tutorials all night. It has helped me get to where I am. I tried folllowing these steps for different modules and it didn’t seem to work for me

anhanyoung

What does Expected ‘]’ to match ‘[’ from line 45 and instead saw ‘:’.
75 Expected ‘}’ to match ‘{’ from line 28 and instead saw 'MMM-lyft mean?

sdetweil

@anhanyoung means u are missing comma, " or something… confused the parser…