Weather forecast only displays 'invalid date'

[justjim1220]

@canterrain

2 things to try…

try putting a comma after the motion detection module

module: ‘MMM-Motion-Detection’,

[Canterrain]

No luck adding the comma unfortunately.

You mentioned 2 things to try, what was the second?

Thanks for looking at this.

[yawns]

appid in currentweather module is missing a ’

Time and date on your raspberry are fine, right?

[justjim1220]

@canterrain

Not sure how I missed that… weird, I swear I put it in there…

Check the time and date of your devices…

[Canterrain]

Thanks for the helpful suggestions.

Date and time are correct on my pi, and the mirror even shows the right date and time.

The config did have the missing ’ in current weather, I apparently accidentally deleted it when I was removing personal info. I edited the above to reflect the current status of my config file.

This was working fine before. About the time I updated the mirror to latest version and added the motion detection module. But it didn’t happen immediately after those events. It was a few days later. I did find a logged issue (that was closed) describing the same thing, after updating the mirror.

[Bob]

I’m having the same issue as @Canterrain

Snipp from Clock and Weather modules are below

 // Clock
		{
			module: "clock",
			position: "top_center",
			config: {
				displayType: "digital",
				displaySeconds: false,
				showPeriod: true,
				showPeriodUpper: false,
				clockBold: false,
				showDate: true,
				showWeek: false,
				dateFormat : "dddd, MMMM Do YYYY",
			},
		},

// Calendar// Weather - ID from http://www.openweathermap.org/help/city_list.txt
		{
			module: "weatherforecast",
			position: "top_left",
			header: "7 Day Forecast for ",
			config: {
				fade: false,
				maxNumberOfDays: 7,
				showRainAmount: true,
				colored: true,
				locationID: "2655603",
				appid: "xxx"
			},
		},

Issue only occurred once I did an update to the MM system to v.2.4.0

[Richard238]

Same issue here

[killing joke]

same here.

Is there a possibilty to erase the comma and country code behind the location in the header?!

[AgP42]

Hello,

same problem for me since the update in 2.4.0… So general update issue probably (looking forward for a solution !:slightly_smiling_face: )

To answer @killing-joke said in Weather forecast only displays ‘invalid date’:

same here.

Is there a possibilty to erase the comma and country code behind the location in the header?!

I did change the following on the file “/MagicMirror/modules/default/weatherferecast/weatherforecast.js” :

// Override getHeader method.
getHeader: function() {
if (this.config.appendLocationNameToHeader) {
//line to change
//return this.data.header + " " + this.fetchedLocationName;
return "Prévisions Météo Nogent"; // write here what you want...
}

return this.data.header;
}, 

Here is the result :

You can of course change the line on the way you want to remove only the part you need.

[killing joke]

@agp42 said in Weather forecast only displays ‘invalid date’:

I did change the following on the file “/MagicMirror/modules/default/weatherferecast/weatherforecast.js” :

Thanks a lot. I got it.

I found a solution on https://github.com/MichMich/MagicMirror/commit/0e2e8d2e2aca2f8d0f1f2eb7b213577fcc4f623a
But I had to change the lines 338 and 339 instead of 336 and 337

var day = moment(forecast.dt_txt, "YYYY-MM-DD hh:mm:ss").format("ddd");
var hour = moment(forecast.dt_txt, "YYYY-MM-DD hh:mm:ss").format("H");

into

var day;
var hour;
if(!!forecast.dt_txt) {
	day = moment(forecast.dt_txt, "YYYY-MM-DD hh:mm:ss").format("ddd");
	hour = moment(forecast.dt_txt, "YYYY-MM-DD hh:mm:ss").format("H");
} else {
	day = moment(forecast.dt, "X").format("ddd");
	hour = moment(forecast.dt, "X").format("H");
}

It works for my MM.