MagicMirror-Documentation icon indicating copy to clipboard operation
MagicMirror-Documentation copied to clipboard

Update compliments.md

Open 3DPixel opened this issue 2 years ago • 8 comments

In the original guide, adding the compliments to the MagicMirror/config/config.js file didn't override the default compliments. By adding them to the defaults file, it works

3DPixel avatar Sep 30 '23 18:09 3DPixel

adding the compliments to the MagicMirror/config/config.js file didn't override the default compliments.

Tested with new compliments in config.js, it's works I think you have made an error in your personal config

There is maybe an error of understanding, I will clarify the doc

sample there

maybe it's better:

{
    module: "compliments",
    position: "lower_third",
    config: {
      compliments: {
        anytime: [
          "Hey there sexy!"
        ],
        morning: [
          "Good morning, handsome!",
          "Enjoy your day!",
          "How was your sleep?"
        ],
        afternoon: [
          "Hello, beauty!",
          "You look sexy!",
          "Looking good today!"
        ],
        evening: [
          "Wow, you look hot!",
          "You look nice!",
          "Hi, sexy!"
        ],
        "....-01-01": [
          "Happy new year!"
        ]
      }
    }
}

@rejas: what do you think about this ?

bugsounet avatar Sep 30 '23 21:09 bugsounet

@bugsounet is right, the moduel works as expected, no need to change the defaults. happy to get a PR for this to clear up

rejas avatar Oct 01 '23 05:10 rejas

I edited the config/compliments.js file and ran the npm check utility and everything checked out but none of the custom compliments worked and the defaults ran instead.

From: Bugsounet - Cédric @.> Sent: Saturday, September 30, 2023 10:08 PM To: MichMich/MagicMirror-Documentation @.> Cc: 3DPixel @.>; Author @.> Subject: Re: [MichMich/MagicMirror-Documentation] Update compliments.md (PR #188)

adding the compliments to the MagicMirror/config/config.js file didn't override the default compliments.

Tested with new compliments in config.js, it's works I think you have made an error in your personal config

There is maybe an error of understanding, I will clarify the doc

sample therehttps://develop.docs.magicmirror.builders/modules/compliments.html#default-value

maybe it's better:

{

module: "compliments",

position: "lower_third",

config: {

  compliments: {

    anytime: [

      "Hey there sexy!"

    ],

    morning: [

      "Good morning, handsome!",

      "Enjoy your day!",

      "How was your sleep?"

    ],

    afternoon: [

      "Hello, beauty!",

      "You look sexy!",

      "Looking good today!"

    ],

    evening: [

      "Wow, you look hot!",

      "You look nice!",

      "Hi, sexy!"

    ],

    "....-01-01": [

      "Happy new year!"

    ]

  }

}

}

@rejashttps://github.com/rejas: what do you thiink about this ?

— Reply to this email directly, view it on GitHubhttps://github.com/MichMich/MagicMirror-Documentation/pull/188#issuecomment-1741859616, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGKWOMG7CVWGADLOAMEMUUDX5CCZHANCNFSM6AAAAAA5NXFMLI. You are receiving this because you authored the thread.Message ID: @.@.>>

3DPixel avatar Oct 02 '23 10:10 3DPixel

Configuration tested (in config/config.js)

		{
			module: "compliments",
			position: "lower_third",
			animateIn: "bounceInDown",
			animateOut: "bounceOutUp",
			config: {
				updateInterval: 500,
				fadeSpeed: 200,
				compliments: {
			          anytime: [
			            "anytime1",
			            "Really ?",
			            "there is not default!!!"
			          ],
			          morning: [
			            "morning1",
			            "morning2",
			            "morning3"
			          ],
			          afternoon: [
			            "afternoon1",
			            "afternoon2",
			            "afternoon3"
			          ],
			          evening: [
			            "evening1",
			            "evening2",
			            "evening3"
			          ],
			          "....-01-01": [
			            "!!!! Happy new year !!!!!"
			          ]
			       }
			}
			
		},

Result:

ezgif com-gif-maker

That you can see, there is no default and the module works as expected. I just speed up interval of update

bugsounet avatar Oct 02 '23 16:10 bugsounet

I edited the config/compliments.js file

Strange filename, you should edit config/config.js

rejas avatar Oct 02 '23 18:10 rejas

what is the content beside the unwanted filename change? It's hard to see, looks like only format has changed (which will be reverted when lintered).

khassel avatar Oct 02 '23 18:10 khassel

Apologies, it was the config/config.js I typed ‘compliements.js’ as I’d not got the details to hand and thought it was the right file name.. of the top of my head.

Back to the original topic. I don’t know why it didn’t work for me, I had just copied the template from the documentation, ran the config checker and it checked out but didn’t work.

As the file mentioned previously did work, is there a reason that there needs to be 2 files that do the same thing? Thanks 😊

From: Veeck @.> Sent: Monday, October 2, 2023 7:01 PM To: MichMich/MagicMirror-Documentation @.> Cc: 3DPixel @.>; Author @.> Subject: Re: [MichMich/MagicMirror-Documentation] Update compliments.md (PR #188)

I edited the config/compliments.js file

Strange filename, you should edit config/config.js

— Reply to this email directly, view it on GitHubhttps://github.com/MichMich/MagicMirror-Documentation/pull/188#issuecomment-1743492699, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGKWOMGG77YLSH2LSXWEHH3X5L6N5AVCNFSM6AAAAAA5NXFMLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBTGQ4TENRZHE. You are receiving this because you authored the thread.Message ID: @.@.>>

3DPixel avatar Oct 02 '23 22:10 3DPixel

Sorry, what is ‘lintered’, is that GitHub lingo? I’m new to GitHub so just getting to grips with it. Thanks

From: Karsten Hassel @.> Sent: Monday, October 2, 2023 7:24 PM To: MichMich/MagicMirror-Documentation @.> Cc: 3DPixel @.>; Author @.> Subject: Re: [MichMich/MagicMirror-Documentation] Update compliments.md (PR #188)

what is the content beside the unwanted filename change? It's hard to see, looks like only format has changed (which will be reverted when lintered).

— Reply to this email directly, view it on GitHubhttps://github.com/MichMich/MagicMirror-Documentation/pull/188#issuecomment-1743523217, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGKWOMCFTDASXBPC45FYJFDX5MBCXAVCNFSM6AAAAAA5NXFMLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBTGUZDGMRRG4. You are receiving this because you authored the thread.Message ID: @.@.>>

3DPixel avatar Oct 02 '23 22:10 3DPixel