Is there a way to show specific compliments on specific dates?

CyanKali

as you can see I don´t know much about coding, but have you tried just deleting all the default compliments and then adding the line that romain suggested?

romain

@CyanKali It should have replace all compliment. At that particular date.
well, yesterday was the 26 right ? you let the 27 so I guess it just wasn’t the rigth date to activate.
Try changing the date. (well, today must be the 27 so It should work today ahah).

@pauabaer you could use the compliment module. add the thing I had and then go on the config.js file and replace all the compliments by empty strings so it will display nothing when not the date

                {                                                                                                                                                                                                                            
                        module: "compliments",                                                                                                                                                                                               
                        position: "lower_third",                                                                                                                                                                                             
                        header: "",                                                                                                                                                                                                          
                        config: {                                                                                                                                                                                                            
                                compliments: {                                                                                                                                                                                               
                                        anytime: [ "" ],                                                                                                                                                                                                   
                                        morning: [ "" ],                                                                                                                                                                                                   
                                        afternoon: [ "" ],                                                                                                                                                                                                   
                                        evening: [  "" ]                                                                                                                                                                                                    
                                }                                                                                                                                                                                                            
                        }                                                                                                                                                                                                                    
                },

Or you could change the code so the module hide itself when the date isn’t right

        // Override dom generator.                                                                                                                                                                                                           
        getDom: function() {
            var complimentText = this.randomCompliment();
            if (moment().format('MMM D') == 'Apr 27')
            {
                this.show();
                complimentText = "Happy birthday"
            }
            else
                this.hide();
            var compliment = document.createTextNode(complimentText);
            var wrapper = document.createElement("div");
            wrapper.className = this.config.classes ? this.config.classes : "thin xlarge bright";
            wrapper.appendChild(compliment);

                return wrapper;
        },

pauabaer

thanks @romain
i think the second way will be mine. what do i have to write in the config.js file? how can i integrate a second date?

romain

@pauabaer for the config file, you can just use the default one. we didn’t change much with that code.
To add another date, just add as many else if () statement as needed. This isn’t a pretty solution but it will work.

        // Override dom generator.                                                                                                                                                                                                           
        getDom: function() {
            var complimentText = this.randomCompliment();
            if (moment().format('MMM D') == 'Apr 27')
            {
                this.show();
                complimentText = "Happy birthday"
            }
         else if (moment().format('MMM D') == 'Apr 28') {
                 this.show();
                complimentText = "Happy 28th day"

         }  else
                this.hide();
            var compliment = document.createTextNode(complimentText);
            var wrapper = document.createElement("div");
            wrapper.className = this.config.classes ? this.config.classes : "thin xlarge bright";
            wrapper.appendChild(compliment);

                return wrapper;
        },

It’s not a pretty solution. I could made it prettier by creating a separate function with a while abd set a variable in the config section so you can add as many date as you want by editing the config.js. But I’m lazy so … yeah, not pretty solution.

pauabaer

yeah @romain. sounds for me as if this solution is very easy at all. you dont blow up the code with two more else if-statements.

But with a separate function you could pimp up the module.
Think about annual festivals like:

eastern --> Happy Eastern
christmas --> Merry Christmas
birthdays --> Happy Birthday

etc.

Perhaps you can overcome the laziness :grin:

romain

@pauabaer Na, perhaps you can make it youself ;)
It’s not that hard, just read about array and function in javascript.
The idea is :
-You creat a function.
-You creat a variable in the config part. the variable should be an array. The easiest thing would be to make it an array of dictionnaries. So you can put a date and a message in each ones.
-Inside the function you stock the value of moment().format(‘MMM D’) in a variable
-Then you make a for that iterate on each element of the array an compare the value of moment().format(‘MMM D’) that you just got to the date part of each dictionary.
-If there is a match, you return the associate message. if not, you just return an empty string.
-In the getDom function you call that function instead of the randomCompliment

And you should be fine.
And hey, if you succeed you’ll could be proud to say you did a module yourself (sort of) !

pauabaer

@romain sounds logic. but i dont have any knowledge about javascript. so i have to start from scratch. i dont know…i think for now your workaround sounds best :D

strawberry 3.141

@pauabaer as you dont have coding experience I guess you have neither experience with git. By modifying the files you will not be able to easily update the magicmirror.

pauabaer

@strawberry-3-141 you are right. i have read about this and i am aware of that fact. means a default function would be better. so i hope @romain can overcome the laziness :sunglasses:

ps @romain i hope you know how i meant this. you do what you want to do. because i dont have any coding experience i can only speak out some wishes/ features to make this project (for more people) a bit more “magically”.

romain

@pauabaer sure, however, I have other projects to work on so I rather focus on them. I don’t have that much knowledge in javascipt either. (I do have some knowledge in other language though). When I do stuff it’s a lot a trial and error until that work (even for the tiny bit of code I made in this post required some research from me. Not a lot of it… but some. I never used the moments() function before)
This is your opportunity to gain knowledge.