MMM-Carousel not displaying

[sdetweil]

@pastormingle don’t know why you would do that. it has to run under MagicMirror…

[mumblebaj]

@pastormingle Do you see any of the module suspended or module resumed stuff?

[sdetweil]

@pastormingle @mumblebaj

ok, I just debugged this, and there is a MM design changed that impacts this module… and maybe others…

if there is NO ‘position’ set, then the initial getDom() is not called. and this module DEPENDS on that
note that without position its getDom() is NEVER called…

so, set

position: ‘bottom_bar’

in the MMM-Carousel module config

[pastormingle]

@sdetweil said in MMM-Carousel not displaying:

position: ‘bottom_bar’

so that would be in the config.js file or the www-Carousel.js file?

[sdetweil]

@pastormingle ALL config options ALWAYS go in config.js

NEVER edit the module source file…

the design is options in config.js override any options defined in the module
any options not USED in config.js use the defaults provided by the module

but position is a MM config option (outside the config:{} block for the module),
NOT a module config option (inside the config:{} block for module)

see
https://docs.magicmirror.builders/modules/configuration.html#example

[sdetweil]

@pastormingle

{
     module:   'MMM-Carousel',
     position: 'bottom_bar',
     config: { 
        whatever config for this module
    }
}

[pastormingle]

@sdetweil ```

So now is gets an Undefined displayed at the bottom of the display.. 
},
        {
            module: 'MMM-Carousel',
             position: 'bottom_bar',
            config: {
                transitionInterval: 10000,
                ignoreModules: [],
                mode: 'global'
            }
        }

![45d7e7ab-536a-4ba1-b849-c0ba73a9344f-image.png](/assets/uploads/files/1638977941062-45d7e7ab-536a-4ba1-b849-c0ba73a9344f-image.png)

[pastormingle]

@pastormingle getting the following error in the dev windows now.
TypeError: Failed to execute ‘appendChild’ on ‘Node’: parameter 1 is not of type ‘Node’.
at updateModuleContent (main.js:215)
at main.js:142
at new Promise ()
at updateDomWithContent (main.js:140)
at main.js:122

[sdetweil]

@pastormingle this is my config

        {
            module: 'MMM-Carousel',
            position: "bottom_bar",
            config: {
                transitionInterval: 10000,
                ignoreModules: ['clock', 'alert'],
                mode: 'slides',
                slides: [
                    ['calendar', 'compliments', 'weather'],
                    ['newsfeed'],
                ]
            }
        }

but you wanted slides mode… why are u set to global?

[sdetweil]

@pastormingle

I changed ‘slides’ to ‘global’ and it worked too

you are using this version

https://github.com/shbatm/MMM-Carousel.git 

in the MMM-Carousel folder do

git remote -v

[pastormingle]

@sdetweil said in MMM-Carousel not displaying:

git remote -v
same.

[mumblebaj]

@pastormingle So, what @sdetweil meant was that in the terminal window in the ~/MagicMirror/modules/MMM-Carousel/ folder you should run the command git remote -v and post the output of that.

For me it is:

origin  https://github.com/barnabycolby/MMM-Carousel (fetch)
origin  https://github.com/barnabycolby/MMM-Carousel (push)

[sdetweil]

@mumblebaj hm… that one also works… (but works without position, but doesn’t have a getDom function)
the sabatm version has page indicator buttons u can push too…

[mumblebaj]

@sdetweil Ah, now I understand the earlier differences. We were looking at different modules. Which one is @pastormingle using?

[sdetweil]

@mumblebaj still waiting for that info

[mumblebaj]

@sdetweil Unless there is a 3rd one that we are not aware of…:face_with_monocle:

[sdetweil]

@mumblebaj yeh, I posted a fix to the one I was using, after it stopped working on 2.14
https://github.com/shbatm/MMM-Carousel/issues/31

[mumblebaj]

@sdetweil And there is this one as well:
https://github.com/schramkef/MMM-Carousel

[sdetweil]

@mumblebaj yep, thats the newest… 9months old, shbatm is 3 years, and the barnaby is 5 years old

[mumblebaj]

@sdetweil Lets see which one he is using. I am off to bed. Been a long day.