Version 2.30.0 weather module, stuck loading, worked fine before upgrade

[Richard238]

@sdetweil

openweathermap.js is unchanged

WeatherProvider.register("openweathermap", {


	providerName: "OpenWeatherMap",

	// Set the default config properties that is specific to this provider
	defaults: {
		apiVersion: "3.0",
		apiBase: "https://api.openweathermap.org/data/",
		// weatherEndpoint is "/onecall" since API 3.0
		// "/onecall", "/forecast" or "/weather" only for pro customers
		weatherEndpoint: "/onecall",
		locationID: false,
		location: false,
		// the /onecall endpoint needs lat / lon values, it doesn't support the locationId
		lat: 0,
		lon: 0,
		apiKey: ""
	},


config.js shows

	{
		module: 'weather',
		header: 'Current zzzzzzzzzzzzzzzz  weather: ',			
		position: 'top_right',
		config: {
			//weatherProvider: "openweathermap",
			providerName: "OpenWeatherMap",
			type: "current",
			lat: nnnnnnnnnnn,
			lon: -nnnnnnnnnnnnn,
			apiKey: "aaaaaaaaaaaaaaaaaaaaaaaaaaa"
			//type: 'current',
			//location: 'xxxxxxxxxxxxxx',
			//locationID: 'zzzzzzzz',  //ID from https://www.openweathermap.org
			//apiKey: 'aaaaaaaaaaaaaaaaaaaaaaaaaaa'
		}
	},

[sdetweil]

@Richard238

providerName: “OpenWeatherMap”

That is not the correct property name

From the doc
https://docs.magicmirror.builders/modules/weather.html#usage

[Richard238]

@sdetweil

With no caps? Still doesn’t work.

config: {
				weatherProvider: "openweathermap",

[sdetweil]

@Richard238 again show the whole config, xxx out the api key

Start w
position
and config
provider
type ( default is current)
apikey
lat
lon

the 3.0 apikey requires a credit card backing, right?

[Richard238]

@sdetweil

		{
			module: 'weather',
			header: 'Current zzzzzzzzzzzzz  weather: ',			
			position: 'top_right',
			config: {
				weatherProvider: "openweathermap",
				lat: 111111111111111, 
				lon: 1111111111111111111111,
				apiKey: "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaa",
				type: 'current',
				units: 'config.units'
				//location: 'zzzzzzzzzzzzzzz',
		}
		},

I must have had that API key for aeons, don’t ever remember it having a c/card behind it.
Maybe it’s out-of-date.

[sdetweil]

@Richard238 yes they told us the old free api was being terminated. only the new api exists, so we changed the defaults

but other users have reported the 2.5 apikey still works
see the 1st post on this topic

[Rberry91]

@sdetweil hi Sam,

I have added a credit card and added the extra codes from the article and it doesn’t load.

[sdetweil]

@Rberry91 if you have the 3.0 apikey you dont add any extra parms

just type, apikey, lat and lon

[Rberry91]

@sdetweil
It still says invalid date

Sorry so many questions

[sdetweil]

@Rberry91 open the developers window, ctrl-shift-i,
select the console tab,
and enter weath in the filter field

do you see errors? like 401, 403

[Rberry91]

@sdetweil
Hi yes error 401

[sdetweil]

@Rberry91 401 means the apikey is invalid

[Rberry91]

@sdetweil
Hi Sam,

So I got it to work however the header is Australia/Brisbane, which is not correct. But the weather details are correct, how do I change the name of the header. I have added location and locationID

[sdetweil]

@Rberry91 v3 does not return the right location,
location/id are not used in v3, only lat/lon

use the module header property

module:…
header:“…”

the set the weather config option

appendLocationToHeader:false,

[Rberry91]

@sdetweil
Hi Sam,

Now I have an message to update the MMM-GooglePhotos module. How do I do that without causing problems like before. I did try your code and it didn’t seem to work but I am most likely doing something incorrect

[sdetweil]

@Rberry91 didnt try my code? i do not supply anything that will update modules.

my upgrade script is only for base MagicMirror

every module upgrade is the same

cd ~/MagicMirror/modules/modulename
git pull
xxx see the module doc on what to do after
sometimes you need an npm install, sometimes not

[Rberry91]

@sdetweil

Hi again,
The MMM- GoohlePhotos was not updating so I re installed and am know getting a error of missing creditionals please help

[sdetweil]

@Rberry91 now you have to follow the module instructions to get authorization again

[Rberry91]

@sdetweil
I reverted to the older version and I have tried updating using your command and other ones and still doesn’t update tell me that “pulling without specifying how to reconcile divergent branches is etc”

The other problem I am having is the weather, the forecast works but the current Weather doesn’t and they are the same code.

[sdetweil]

@Rberry91 divergent branches sounds like you have both modules trying to merge to the same folder

delete then git clone again
the nan says the apikey is wrong

and you have a typo, MagicMirror is case sensitive