MMM-Tube-Status (London Underground)

[djbenny07]

yes, it is the entire file it currently works.

[nigel-daniels]

@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.

[nigel-daniels]

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]

OK I have found that i didn’t copy all the text. the below does not work

’

var config = {
port: 8080,

    language: 'en',
    timeFormat: 24,
    units: 'metric',

    modules: [
            {
                    module: 'alert',
            },
            {
                    module: 'clock',
                    position: 'top_left'
            },
            {
                    module: 'calendar',
                    header: 'Public Holidays',
                    position: 'top_left',
                    config: {
                            calendars: [
                                    {
                                            symbol: 'calendar-check-o ',
                                            url: 'webcal://www.calendarlabs.com/templates/ical/UK-Holidays.ics'
                                    }
                            ]
                    }
            },
            {
                    module: 'compliments',
                    position: 'lower_third'
            },
            {
                    module: 'currentweather',
                    position: 'top_right',
                    config: {
                            location: 'London',
                            locationID: '264443',  //ID from http://www.openweathermap.org
                            appid: 'number'
                    }
            },
            {
                    module: 'weatherforecast',
                    position: 'top_right',
                    header: 'Weather Forecast',
                    config: {
                location: 'London',
                            locationID: '264443',  //ID from http://www.openweathermap.org
                appid: 'Number'
                    }
            },
            {
            module: 'MMM-Tube-Status'
            position: 'top_left'
            header: 'Tube'
            }
            {
                    module: 'newsfeed',
                    position: 'bottom_bar',
                    config: {
                            feeds: [
                                    {
                                            title: "BBC Top Stories",
                                            url: "http://feeds.bbci.co.uk/news/rss.xml"
                                    }
                            ],
                            showSourceTitle: true,
                            showPublishDate: true
                    }
            },
    ]

};

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

[strawberry 3.141]

@djbenny07 you missed 3 commas in there

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

[nigel-daniels]

@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.

[djbenny07]

can the position be the same as another module though? i still get the message to saying ‘Please create config file’

as my clock is in top_left

EDIT: Just noticed that i missed the ’ from the module name. added that in and still issue

[yawns]

@djbenny07 you are still missing the closing , at the end of the tube module lines, as several users mentioned above

[djbenny07]

@yawns yes my bad. i noticed this and corrected it. still same message

[yawns]

@djbenny07 are you kidding? Look at the three lines you got there for the tube module. All three lines MUST end with a comma