Update MMM-ModuleScheduler.js to work with non-visible modules

ianperrin / MMM-ModuleScheduler

A MagicMirror helper module to schedule the display of modules and sending of notifications.

MIT License

101 stars 13 forks source link

Update MMM-ModuleScheduler.js to work with non-visible modules #27

Closed cschwarz007 closed 3 years ago

cschwarz007 commented 3 years ago

I have some modules (such as MMM-mmpm and others) that do not show on the screen but are in the list of modules. Whenever ModuleScheduler tries to show / hide these modules, it runs into an error and aborts. Hence this fix to allow for these type of modules and have ModuleScheduler work correctly only for visible modules.

ianperrin commented 3 years ago

@cschwarz007 thanks for the pull request. The core module file has changed following the user of prettier to format the code. As a result the pull request has conflicts. Can you update, fix and resubmit? Thanks

ianperrin commented 3 years ago

Applied fix in fb894ae https://github.com/ianperrin/MMM-ModuleScheduler/blob/fb894aef37205f5da8562182ffa51f6655f0365d/MMM-ModuleScheduler.js#L138-L141