MagicMirror Forum
    • Recent
    • Tags
    • Unsolved
    • Solved
    • MagicMirror² Repository
    • Documentation
    • 3rd-Party-Modules
    • Donate
    • Discord
    • Register
    • Login
    A New Chapter for MagicMirror: The Community Takes the Lead
    Read the statement by Michael Teeuw here.

    MMM-Tube-Status (London Underground)

    Scheduled Pinned Locked Moved Transport
    25 Posts 5 Posters 20.4k Views 6 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • N Offline
      nigel-daniels @djbenny07
      last edited by

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

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

      1 Reply Last reply Reply Quote 0
      • D Offline
        djbenny07
        last edited by 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

        0_1477936546318_config.png

        yawnsY 1 Reply Last reply Reply Quote 0
        • yawnsY Offline
          yawns Moderator @djbenny07
          last edited by yawns

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

          D 1 Reply Last reply Reply Quote 0
          • D Offline
            djbenny07 @yawns
            last edited by

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

            0_1477936934408_Screenshot_5.png

            yawnsY 1 Reply Last reply Reply Quote 0
            • yawnsY Offline
              yawns Moderator @djbenny07
              last edited by

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

              D 1 Reply Last reply Reply Quote 1
              • D Offline
                djbenny07 @yawns
                last edited by

                @yawns haha, im being really dumb i know. i have finally seen it, long day. thats what im going to blame it on.

                its working now!

                N 1 Reply Last reply Reply Quote 1
                • yawnsY Offline
                  yawns Moderator
                  last edited by

                  Sometimes a knock at the back of ones head is worth a try.

                  Good to see it’s finally working :-)

                  1 Reply Last reply Reply Quote 0
                  • N Offline
                    nigel-daniels @djbenny07
                    last edited by

                    @djbenny07 Yeay! It lives :)

                    1 Reply Last reply Reply Quote 0
                    • D Offline
                      djbenny07
                      last edited by

                      i presume to update the module i just run this again?

                      git clone https://github.com/nigel-daniels/MMM-Tube-Status.

                      yawnsY 1 Reply Last reply Reply Quote 0
                      • yawnsY Offline
                        yawns Moderator @djbenny07
                        last edited by

                        @djbenny07
                        enter the MMM-Tube-Status folder and run git pull, that should pull all changes from github
                        If you run the clone command again it will tell you the target directory already exists

                        1 Reply Last reply Reply Quote 0
                        • 1
                        • 2
                        • 3
                        • 2 / 3
                        • First post
                          Last post
                        Enjoying MagicMirror? Please consider a donation!
                        MagicMirror created by Michael Teeuw.
                        Forum managed by Sam, technical setup by Karsten.
                        This forum is using NodeBB as its core | Contributors
                        Contact | Privacy Policy