MMM-NOAA - Another Weather Module

[SoleLo]

@mykle1 I have one commented out, I’m going to turn it on or just wait…

[nobita]

@justjim1220
Work now,it’s my mistake i use old API Key
Thanks again

[redink]

I know you are working on V3, and I am looking forward to it. But, I’ve been having issue with V2. Specifically, it wouldn’t work with MMM-CalendarExt. Meaning I couldn’t have both on screen at the same time.

I looked at the errors, and the issue appears to be with the following lines of code

 	if (this.config.units != "metric") {
	 tt.innerHTML = n Day ~ " + allDay[0].fcttext + "<br />" 
	 	: "Night ~ " + allDay1 + "<br />";
	 	} else {
	 tt.innerHTML = n " + allDay[0].fcttext_metric + "<br />" 
	 	: "" + allDayM1 + "<br />";  
	 	}

The error was something about an uncaught exception from a promise. My JavaScript is very rusty to actually try and fix this on my own, but simply commenting out those lines actually made the error go away, and I was able to get NOAA and CalendarExt on screen at the same time.

Just posting this because I spent the last two days trying to get this resolved, and figured someone else might benefit.

[cowboysdude]

@solelo said in MMM-NOAA - Another Weather Module:

@mykle1 I can tell there is a lot of love in that “maniac”, he is just trying to tease us.

[SoleLo]

@cowboysdude can we have the V3… you killing us…

[cowboysdude]

@solelo Working out some things still ;) Hang in there

[Mykle1]

@cowboysdude said in MMM-NOAA - Another Weather Module:

Working out some things still Hang in there

We want the world and we want it now. :-)

[justjim1220]

@cowboysdude

Yeah, what Mykle1 said… Now dammit! :face_with_stuck-out_tongue_closed_eyes:

[cowboysdude]

John Isabella: I’m truly sorry about your brother. It’s too bad.

Truman Gates: Oh no sir. You ain’t seen bad yet, but its coming.

:face_with_tears_of_joy:

[cowboysdude]

@redink Well 3 is close…so if you have issue with that PLEASE let me know… It should be by this weekend…