How can I add more time periods for MMM-compliments?

DæmonEyes

This is actually my first time programming ever, so I don’t actually know what are logic or switch functions.

This is what I attempted, but it didn’t work, and instead the compliments module just didn’t load up when starting MM:

![I added two more time period: predawn and night](https://imgur.com/a/JJrJ01w)

![Updated the time intervals](https://imgur.com/a/lzAO0gJ)

![And tried modifying the compliments array as well](https://imgur.com/a/ZiNT8u5)

sdetweil

@dæmoneyes your changes didn’t show
use the three back-tics before and after your code
on keyboard, left of the number 1 key

DæmonEyes

@sdetweil Here are all the changes I tried: https://imgur.com/a/wbYQclq
alt text

sdetweil

@dæmoneyes start the mirror with

npm start dev

and in the white developers window, select the console tab,
scroll up/down to find any errors (red text)…

I can’t see all your code, but its probably a missing quote, paren or brace

DæmonEyes

@sdetweil Did that, found some red related to compliments and also to openweather
Here is the full console tab: https://imgur.com/a/Lrn8kVH

sdetweil

@dæmoneyes compliments.js:48 syntax error

that means on line 48 of the complements.js file the compiler found an error…

DæmonEyes

@sdetweil Once again, I owe you my thanks sdetweil! It works now!

What should I do about the red related to the openweathermap?

sdetweil

@dæmoneyes openweathermap… I do not know… sorry…

DæmonEyes

@sdetweil hey no problem, it seems to be working fine, so I guess I shouldn’t worry about it.

If I may ask one more question, should I do anything about the yellow warning ‘Electron Security Warning’?

sdetweil

@dæmoneyes if your mirror was open to the internet, then I would say yes

DæmonEyes

@sdetweil Hmm ok, besides weather and quote of the day, I’m not really connecting to the net for anything so I guess that’s fine.
Thanks again for helping me solve this!

SaschaSP

@DæmonEyes you only make changes in
MagicMirror/modules/default/compliments/MMM-Compliments.js
and of corse in the
MagicMirror/config/config.js?

Can your Post your changes here in the Forum?
Here is a nice “HowTo” post a code into the Forum for better reading https://forum.magicmirror.builders/topic/4247/how-to-post-code-on-the-forum-for-absolute-beginners

Thank you

DæmonEyes

@saschasp Actually I only had to make changes in the MMM-compliments.js and not the config.js
I just had to make changes at those 3 places that I showed in the imgur link I had posted and it works great.
sdetweil helped my find the missing comma that was causing the problem with the npm start dev prompt and now it works great.

I now have 5 time periods and 3 compliments for each period. If I think of more compliments, I’ll probably add a 4th compliment for each period.

DæmonEyes

Is it normal for the default modules to reset to factory settings when updating MM?
Just did the MM update to 2.7.1 and noticed my compliments module has reset to its original compliments and time periods.

sdetweil

yes, the default modules are part of the base… and will be reset… altho you should have been warned by git

if u did a git stash, then u can get those changes back

DæmonEyes

@sdetweil Oh ok, I just did my first stash now.
So let’s say I had done a stash before I had updated; how would I retrieve it after updating?

(So I know what to do next time)

sdetweil

@DæmonEyes git stash pop

i don’t know if you can pop one file at a time

(ps… google is your friend!)