Show calendar based on IP Address accessed

[sdetweil]

@Spinster another conversation highlighted another change required
and a bug

I’ll update my code here later

oh, and to help you, you could fork the MagicMirror repo
and use it, adding this code,
and saving to the repo, so next update
you can sync the fork and git pull to your system

sorry my new phone and thumb don’t agree where the keys are… lol and correction never gets it right

[Spinster]

@sdetweil

I can’t thank you much for your sincerity in making it correct. Looking forward to the updated code.
I will try to contribute too in future like you.
Thank you once again.

[sdetweil]

@Spinster

SO, the ‘bug’ is that this notification , added in pull request 2881 in
https://github.com/MagicMirrorOrg/MagicMirror/pull/2881
in August 2022
edit: I just opened issue 3443 for this problem
https://github.com/MagicMirrorOrg/MagicMirror/issues/3443

	socketNotificationReceived (notification, payload) {
		if (notification === "FETCH_CALENDAR") {
			this.sendSocketNotification(notification, { url: payload.url, id: this.identifier });
		}

added the code in the WRONG place… socketNotificationReceived is ONLY triggered by the node_helper sendSocketNotification,
NOT by other modules doing sendNotification
NO OTHER module can force a sendSocketNotification from our helper

THAT is notificationReceived… (which we JUST added for the 1st time… oops)

So,
remove this

		if (notification === "FETCH_CALENDAR") {
			this.sendSocketNotification(notification, { url: payload.url, id: this.identifier });
		}

and move it to the notificationReceived() function

@sdetweil said in Show calendar based on IP Address accessed:

   notificationReceived(notification,payload){
 	if(notification === 'ALL_MODULES_STARTED'){
        }  else  if (notification === "FETCH_CALENDAR") {
        	this.sendSocketNotification(notification, { url: payload.url, id: this.identifier });
        }

and THEN we need to change the this.identifier to account for the ip based ID, so if some OTHER module
asks THIS calendar instance to refresh ITS data, it asks for the correct content

      else  if (notification === "FETCH_CALENDAR") {
        	this.sendSocketNotification(notification, { url: payload.url, id: this.identifier+(this.ourIPAddress?'_'+this.ourIPAddress:'') });
        }

I’ll add this to the other code, done … I’ll leave the explanation here

[Spinster]

@sdetweil

Perfect, I did this. Thank you so much for your timely help.

Now I am trying to do the same thing in MMM-CalendarEtx2. Because, I want a proper Month Calendar, which is not provided by Calendar.

Is there any module which can display two month calendar, since I searched in modules and could not find.

Thank you once again.

[sdetweil]

@Spinster ext2 is dead. use ext3, it uses the broadcast from the default calendar module which now is just those for this system

uh, hard to put up one wall cal view, 2 I don’t know.

remove position from default cal, it won’t display but will send it broadcast

[Spinster]

@sdetweil

Oh I did not know, but ext3, I don’t see an option to view a month calendar (not month schedule). I am looking for something like MMM-CalendarExtMiniMonth. Please let me know if this option is available

[Spinster]

@sdetweil

Okay will try it. Thank you.

[sdetweil]

@Spinster he has 4 modules, one might provide a view u want

MMRIZE/MMM-CalendarExt3
MagicMirror calendar view module
JavaScript
·
44
·
Updated 10 days ago

MMRIZE/MMM-CalendarExt3Agenda
Daily agenda view module of MagicMirror
JavaScript
·
12
·
Updated 3 days ago

MMRIZE/MMM-CalendarExt3Timeline
Successor of CX2Timeline, Magicmirror module.
JavaScript
·
7
·
Updated on May 4, 2023

MMRIZE/MMM-CalendarExt3Journal
Magic mirror module for presenting events as daily/weekly journal style.
JavaScript
·
3
·
Updated 10 days ago

[Spinster]

@sdetweil

Perfect EXT3 with base calendar is working taking account of IP access settings.

However EXT3 does not have a simple monthly Calendar view like EXT2 Mini month.

Thank you so much for the directions.

[sdetweil]

@Spinster sorry, I don’t know minimonth