Current weather missing

[jeaniouso]

Hi All!

I just installed a transit module last night after feverishly trying to get it to work for hours so, after success with that thanks to two members, I now have another problem. The current weather portion of my MM no longer shows up. I tried disabling the new module however that doesn’t help. I also tried an entire different API and with that variable it’s still the same… Please help… Thanks…```

/* Magic Mirror Config Sample
 *
 * By Michael Teeuw http://michaelteeuw.nl
 * MIT Licensed.
 *
 * For more information on how you can configure this file
 * See https://github.com/MichMich/MagicMirror#configuration
 *
 */

var config = {
	address: "localhost", // Address to listen on, can be:
	                      // - "localhost", "127.0.0.1", "::1" to listen on loopback interface
	                      // - another specific IPv4/6 to listen on a specific interface
	                      // - "0.0.0.0", "::" to listen on any interface
	                      // Default, when address config is left out or empty, is "localhost"
	port: 8080,
	ipWhitelist: ["127.0.0.1", "::ffff:127.0.0.1", "::1"], // Set [] to allow all IP addresses
	                                                       // or add a specific IPv4 of 192.168.1.5 :
	                                                       // ["127.0.0.1", "::ffff:127.0.0.1", "::1", "::ffff:192.168.1.5"],
	                                                       // or IPv4 range of 192.168.3.0 --> 192.168.3.15 use CIDR format :
	                                                       // ["127.0.0.1", "::ffff:127.0.0.1", "::1", "::ffff:192.168.3.0/28"],

	useHttps: false, 		// Support HTTPS or not, default "false" will use HTTP
	httpsPrivateKey: "", 	// HTTPS private key path, only require when useHttps is true
	httpsCertificate: "", 	// HTTPS Certificate path, only require when useHttps is true

	language: "en",
	timeFormat: 12,
	units: "imperial",
	// serverOnly:  true/false/"local" ,
			     // local for armv6l processors, default
			     //   starts serveronly and then starts chrome browser
			     // false, default for all  NON-armv6l devices
			     // true, force serveronly mode, because you want to.. no UI on this device

	modules: [
		{
			module: "alert",
		},
		{
			module: "updatenotification",
			position: "top_bar"
		},
		{
			module: "clock",
			position: "top_left"
		},
		{
			module: "calendar",
			header: "US Holidays",
			position: "top_left",
			config: {
				calendars: [
					{
						symbol: "calendar-check",
						url: "webcal://www.calendarlabs.com/ical-calendar/ics/76/US_Holidays.ics"					}
				]
			}
		},
		  
		{
    module: 'MMM-nyc-transit',
    position: "top_bar",
    header: "Next Train",
    config: {
      apiKey: '632718b7d31a32ec745fd296c6d824a0',
      displayType: 'list',
      mtaType: 'train',
      stations: [
        {
          stationId: 604,
          walkingTime: 5,
          dir: {
              upTown: true,
              downTown: false
          }
        },
        {
          stationId: 604,
          walkingTime: 5,
          dir: {
            upTown: false,
            downTown: true
          }
        }
      ],
      updateInterval: 300000
    }
  },
  
		

		
  
  
		{
			module: "compliments",
			position: "lower_third"
		},
		{
			module: "currentweather",
			position: "top_right",
			config: {
				location: "New York",
				locationID: "5133268 ", //ID from http://bulk.openweathermap.org/sample/city.list.json.gz; unzip the gz file and find your city
				appid: ""
			}
		},
		{
			module: "weatherforecast",
			position: "top_right",
			header: "Weather Forecast",
			config: {
				location: "New York",
				locationID: "5128581", //ID from http://bulk.openweathermap.org/sample/city.list.json.gz; unzip the gz file and find your city
				appid: "884eb65b89072dea57b1be818275ceca"
			}
		},
		{
			module: "newsfeed",
			position: "bottom_bar",
			config: {
				feeds: [
					{
						title: "New York Times",
						url: "http://www.nytimes.com/services/xml/rss/nyt/HomePage.xml"
					}
				],
				showSourceTitle: true,
				showPublishDate: true,
				broadcastNewsFeeds: true,
				broadcastNewsUpdates: true
			}
		},
	]

};

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

[sdetweil]

@jeaniouso said in Current weather missing:

  	config: {
  		location: "New York",
  		locationID: "5133268 ", //ID from http://bulk.openweathermap.org/sample/city.list.json.gz; unzip the gz file and find your city
  		appid: ""

did u put in an appid? you must

[jeaniouso]

@sdetweil I did , I just removed it for privacy reasons.

[sdetweil]

@jeaniouso ok, we typically look to see **** in the field rather than empty… “” may be the default, “****”
is never the default… saves us and u one question cycle

[sdetweil]

@sdetweil did u open the developers window, console tab, and also look at the terminal window or pm2 logs for any error messages

[Mykle1]

@jeaniouso said in Current weather missing:

I now have another problem.

You have a space after the locationID in the currentweather entry, just before the closed quote. Remove the space. It should look like this locationID: "5133268",

[Mykle1]

@jeaniouso

Also, you have 2 different locationID’s. Just thought I would mention that.

[jeaniouso]

@sdetweil Got it … I’m learning i’m learning lol

[jeaniouso]

@Mykle1 Hi There! yes I wanted to get a near accurate location of the same area or at least the immediate surrounding area versus just having the general NYC temp… I kinda wanted to at least get a neighborhood close to weather the temperature might be … Let me fix that… Thanks as ALWAYS

[jeaniouso]

@Mykle1 That DID THE TRICK!!!