CORS policy

[sdetweil]

@bicolorbore586 never edit our code, all config goes in config.js

u need lat and long (how else does it know where u are)

all numbers and true/false have no quotes
anything with a letter or special character does need quotes

u need weatherProvider
apikey
lat
lon
and type (forecast or current)
and

useCorsProxy: true

[bicolorbore586]

@sdetweil
ok, have put it back to how it was… that was final “can I get this to work, before I ask for further help”
this is my config.js

code_text
``` {
			module: "weather",
			position: "top_right",
			config: {
				weatherProvider: "weatherbit",
				type: "current",
				lat: 5,
				lon: -1,
				apiKey: "ab4",
				useCorsProxy: true
					}
		},
		{
			module: "weather",
			position: "top_right",
			header: "Weather Forecast",
			config: {
				weatherProvider: "weatherbit",
				type: "forecast",
				lat: 5,
				lon: -1,
				initialLoadDelay: 1,
				apiKey: "ab4",
				useCorsProxy: true
					}
		},

[sdetweil]

@bicolorbore586 ok, lousy provider error reporting…

i want you to edit the provider weatherbit.js and

using your config (with the bad apikey) I get the ‘0’ error too

change this (starting on line 38)

					if (!data || !data.data[0] || typeof data.data[0].temp === "undefined") {
						// No usable data?
						return;
					}

					const currentWeather = this.generateWeatherDayFromCurrentWeather(data);
					this.setCurrentWeather(currentWeather);

to this

				if(!data || (data && data.error)){
					Log.error("Could not load data ... ", data?data.error: "no data returned");
				}
				else {
					if (!data || !data.data[0] || typeof data.data[0].temp === "undefined") {
						// No usable data?
						return;
					}

					const currentWeather = this.generateWeatherDayFromCurrentWeather(data);
					this.setCurrentWeather(currentWeather);
				}

then save and
restart mm, open the developers window
ctrl-shift-i
select the console tab
and enter weath
in the filter field

there should be a red line of text , with the weatherbit.js name to the right

[bicolorbore586]

@sdetweil
so I commented out the original text and added in the new:

code_text
``` fetchCurrentWeather() {
		this.fetchData(this.getUrl())
			.then((data) => {
				/*if (!data || !data.data[0] || typeof data.data[0].temp === "undefined") {
					// No usable data?
					return;
				}

				const currentWeather = this.generateWeatherDayFromCurrentWeather(data);
				this.setCurrentWeather(currentWeather);*/
				if(!data || (data && data.error)){
					Log.error("Could not load data ... ", data?data.error: "no data returned");
				}
				else {
					if (!data || !data.data[0] || typeof data.data[0].temp === "undefined") {
						// No usable data?
						return;
					}

					const currentWeather = this.generateWeatherDayFromCurrentWeather(data);
					this.setCurrentWeather(currentWeather);
				}
			})
			.catch(function (request) {
				Log.error("Could not load data ... ", request);
			})
			.finally(() => this.updateAvailable());

[bicolorbore586]

@bicolorbore586 from the line Could not load data … TypeError: Cannot read properties of undefined (reading ‘0’) at weatherbit.is

[sdetweil]

@bicolorbore586 can u email me your apikey

same userid as on this post
at gmail

or send me a direct message

the returned data is not coming back in the expected format

[sdetweil]

@bicolorbore586 thanks for the key

the error is

Your request count (159) is over the allowed limit of 50 per day - Upgrade your key, or retry after 670.2 minutes

and u have 2 modules configured, so each will fetch separately, so u have to change the update cycle to once/hour
in both modules, (default 10 mins)

updateInterval: 60 * 60 * 1000

and every MM restart sends 2 requests, 1 for each module

change the new code to look like this…

				if(!data || (data && data.error) || data.status_message){
					Log.error("Could not load data ... ", data?(data.error|| data.status_message): "no data returned");
				}
				else {
					if (!data || !data.data[0] || typeof data.data[0].temp === "undefined") {
						// No usable data?
						return;
					}

					const currentWeather = this.generateWeatherDayFromCurrentWeather(data);
					this.setCurrentWeather(currentWeather);
				}

crappy api implementation… should set error for all errors, not
put error info in good data returns