Not updating at midnight...

MMRIZE

@BKeyport
Your DOM might not be rendered yet on “start” imo. I think manipulating DOM after “DOM_OBJECT_CREATED” notification would be safer.

sdetweil

@MMRIZE

Yes the documented flow is, as I recall on my phone,

Init
Getstyles
Getscripts
GetTemplates
Start
Notification received
AllModulesStarted
DOMobjectsCreated
getDOM

BKeyport

@sdetweil @MMRIZE so, should I move the initial call somewhere else?

sdetweil

@BKeyport after the notification

From the doc

DOM_OBJECTS_CREATED - All dom objects are created. The system is now ready to perform visual changes.

BKeyport

@sdetweil said in Not updating at midnight...:

DOM_OBJECTS_CREATED - All dom objects are created. The system is now ready to perform visual changes

start: function () {
    function scheduleMidnightUpdate() {
        const now = new Date();
        const nextMidnight = new Date(now);

        // Set the time to midnight
        nextMidnight.setHours(24, 0, 0, 0);

        // Calculate the time remaining until the next midnight
        const timeUntilMidnight = nextMidnight - now;

        // Schedule the updateDom method to be called at midnight
        setTimeout(() => {
            this.updateDom();

            // Reschedule the update for the next midnight
            scheduleMidnightUpdate.call(this);
        }, timeUntilMidnight);
    }
},

    notificationReceived: function (notification, payload, sender) {
        if (notification === 'CALENDAR_EVENTS') {
            this.storedEvents = JSON.parse(JSON.stringify(payload))
            this.updateDom();
        } else if (notification === 'DOM_OBJECTS_CREATED') {
            // Call the function to start the first schedule
              scheduleMidnightUpdate.call(this);  
        }

This is the only source I see in the docs of that - a notification. Is that correct?

Thanks for the patience after the double misunderstanding.

sdetweil

@BKeyport

https://docs.magicmirror.builders/development/core-module-file.html

you could have done all the initial event calculating in one function

used interval for the midnights (as it’s a fixed number)

sdetweil

@BKeyport MODULE_DOM_CREATED means YOUR getDom()/getTemplateData() has been called

BKeyport

@sdetweil yeah, doing it that way just throws headers with “Undefined” until it triggers. I’m not gonna go down that rabbit hole. It seems to work as is, so I’m gonna release for now.

sdetweil

@BKeyport said in Not updating at midnight...:

start: function () {
function scheduleMidnightUpdate() {
const now = new Date();
const nextMidnight = new Date(now);

    // Set the time to midnight
    nextMidnight.setHours(24, 0, 0, 0);

    // Calculate the time remaining until the next midnight
    const timeUntilMidnight = nextMidnight - now;

    // Schedule the updateDom method to be called at midnight
    setTimeout(() => {
        this.updateDom();

        // Reschedule the update for the next midnight
        scheduleMidnightUpdate.call(this);
    }, timeUntilMidnight);
}

},

I would do it like this , much more straight forward

     function scheduleMidnightUpdate() {
         const now = new Date();
         const nextMidnight = new Date(now);
 
         // Set the time to midnight
         nextMidnight.setHours(24, 0, 0, 0);
 
         // Calculate the time remaining until the next midnight
         const timeUntilMidnight = nextMidnight - now;
 
         // Schedule the updateDom method to be called at midnight
         setTimeout(() => {
             // trigger first update on next midnight change
             this.updateDom();
            // start Recurring midnight change
             setInterval(()=>{
                 this.updateDom()
             }, 24*60*60*1000 ) // 24 hours * 60 minutes * 60 seconds * 1000 milliseconds (1 day elapsed in ms)  setInterval delay
         }, timeUntilMidnight);  // setTimeout delay
     }
 },

then the system triggers every midnight… not your code