@soqueen hi what does your config look like? If you PM it to me (sans API keys etc.) I’ll try and recreate the issue and suggest a correct config or add a fix to the css.

@StefanBols thanks! I have accepted the pull request.

I have seen a problem when using an iCloud url with the calendar module that I thought I’d post in case anyone else runs into the issue.

Symptoms
The symptom is the calendar sticks with the ‘Loading…’ message but never loads. On the command line I saw the following error:

Ready to go! Please point your browser to: http://localhost:8080
Create new calendar fetcher for url: http://p02-calendarws.icloud.com/ca/subscribe/1/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx - Interval: 300000
Whoops! There was an uncaught exception...
[Error: Invalid UNTIL value: 20160730T115959]
MagicMirror will not quit, but it might be a good idea to check why this happened. Maybe no internet connection?
If you think this really is an issue, please open an issue on GitHub: https://github.com/MichMich/MagicMirror/issues

Cause
With Mitchs support the problem has been traced to the rrule library which handles recurring calendar events. There is an open issue on the library here:

https://github.com/jkbrzt/rrule/issues/145

It seems the library is strictly following RFC2445 detailing how UTC dates work, but it seems not every calendar provider is structuring the dates per RFC2445 when they are set to a local time.

I’ve added the problem I’m seeing to the issue 145. I hope that helps anyone else encountering the same issue.

@cowboysdude np. The two files impacted are:

chancestorms.gif
nt_chancestorms.gif

They should be named:

chancetstorms.gif
nt_chancetstorms.gif

@djbenny07 Yeay! It lives :)

@djbenny07 ok there were a couple issues here that my linter picked up:

1. the tube module must look like this:

{
module: 'MMM-Tube-Status',
position: 'top_left',
header: 'Tube'
},

Note the terminating commas on each line bar the last one in the braces.

2. The final line seems to use ticks to quote undefined not single quotes so it should be:

if (typeof module !== 'undefined') {module.exports = config;}

It can happen that copying between editors, email clients, browsers, etc introduces this sort of error and it’s a real pain to spot!

I hope this fixes it up for you.

FYI there is a minor fix to the chance of storm icons, there was a typo in the gif files name which prevented it from displaying. This also impacted MMM-3Day-Forecast.

FYI there is a minor fix to the chance of storm icons, there was a typo in the gif files name which prevented it from displaying. This also impacted MMM-Weather-Now.

BTW I have just updated the code so that if the optional show_all=false config is used and all of the lines are offering a good service there is a line displayed to indicate this rather than the module just disappearing.

@djbenny07 I noticed that at the end of the file the last two lines should change and it should end something like:

{
module: 'weatherforecast',
position: 'top_right',
header: 'Weather Forecast',
config: {
location: 'London',
locationID: '245613', //ID from http://www.openweathermap.org
appid: 'ID Number taken out'
}
}]
};

/*************** DO NOT EDIT THE LINE BELOW ***************/
if (typeof module !== 'undefined') {module.exports = config;}

If you then want to add the tube status module something like this should work:

{
module: 'weatherforecast',
position: 'top_right',
header: 'Weather Forecast',
config: {
location: 'London',
locationID: '245613', //ID from http://www.openweathermap.org
appid: 'ID Number taken out'
}
},
{
module: 'MMM-Tube-Status',
position: 'top_left',
header: 'Tube Status'
}]
};

/*************** DO NOT EDIT THE LINE BELOW ***************/
if (typeof module !== 'undefined') {module.exports = config;}

I hope that helps.