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

Assassins

@sdetweil
I can only see the header now, where is write Ballerup

{
  module: 'weather',
  position: 'bottom_left',
  header: 'Ballerup',
  config: {
    apiVersion: "2.5",
    weatherEndpoint: "/current",
     lat: "55.7317",
     lon: "12.3633",
     apiKey: "xxxxx765xxxx98xxxjhx888"
   }
},

sdetweil

@Assassins

sorry

weatherEndpoint: “current” or 'forecast" , no /

also, the old api use location or locationID, not lat/lon
and I see you can’t look them up anymore on OWM site

here is from the old doc

locationID	Location ID from OpenWeatherMap This will override anything you put in location.

Leave blank if you want to use location. <------=======

Example: 1234567
Default value: false

Note: When the location and locationID are both not set, the location will be based on the information provided by the calendar module. The first upcoming event with location data will be used.

======================================================================
location	The location used for weather information.

Example: 'Amsterdam,Netherlands'
Default value: false

Note: When the location and locationID are both not set, the location will be based on the information provided by the calendar module. The first upcoming event with location data will be used.

so it ‘seems’ that location by itself would do it.

Assassins

@sdetweil
This is the same, can only see the header.
Is work yesterday on MM 2.29.0

{
  module: 'weather',
  position: 'bottom_left',
  header: 'Ballerup',
  config: {
    apiVersion: '2.5',
   apiBase: 'https://api.openweathermap.org/data/',
    weatherEndpoint: '/no',
    location: 'Ballerup',
    locationID: '2624341',
    apiKey: 'BBnjsdnsdfnsdj8789dsfjsdnfnXX',
    showHumidity: true,
    initialLoadDelay: 1500
  }
},

sdetweil

@Assassins said in Version 2.30.0 weather module, stuck loading, worked fine before upgrade:

apiBase: ‘https://api.openweathermap.org/data/’,
weatherEndpoint: ‘/no’,

no apiBase, that didn’t change

the weatherEndpoint is either “current” OR “forecast”

I originally gave them to you with leading / , so my comment was without the / (no /)

Assassins

@sdetweil
Is the same mate

{
  module: 'weather',
  position: 'bottom_left',
  header: 'Ballerup',
  config: {
    apiVersion: '2.5',
    weatherEndpoint: 'current',    // i have try the forecast to
    location: 'Ballerup',
    locationID: '2624341',
    apiKey: 'xxjljkjsksldjkj3478bjjk',
    showHumidity: true,
    initialLoadDelay: 1500
  }
},

sdetweil

@Assassins seems it doesn’t work anymore… using my old 2.5 key I get code failure with openweathermap

but we changed the sample defaults to a free provider that might work

		{
			module: "weather",
			position: "top_right",
			config: {
				weatherProvider: "openmeteo",
				type: "current",
				lat: 40.776676,
				lon: -73.971321
			}
		},
		{
			module: "weather",
			position: "top_right",
			header: "Weather Forecast",
			config: {
				weatherProvider: "openmeteo",
				type: "forecast",
				lat: 40.776676,
				lon: -73.971321
			}
		}

Assassins

@sdetweil said in Version 2.30.0 weather module, stuck loading, worked fine before upgrade:

@Assassins seems it doesn’t work anymore… using my old 2.5 key I get code failure with openweathermap

Strange, works on my sd card with MM2.29.0 and also works on MMM-DynamicWeather and on MMM-WeatherOrNot with MM2.33.0.
Don’t know why openweathermap doesn’t work with MM2.33.0.
Besides that, it may have something to do with the code in MM2.33.0 that has been rewritten.