Added check to make sure the update interval is at least 10 minutes

Some users get HTTP 429 from the API. This indicates too many request. In the terms of service, yr says "don't make more than one poll every 10 mins".

I wanted to change the config instead of crashing the module, but the setInterval was still called with the original value.

Author: MagMar94

CHANGELOG.md

@@ -21,6 +21,7 @@ planned for 2025-04-01
 - [core] Optimize systeminformation calls and output (#3689)
 - [core] Add issue templates for feature requests and bug reports (#3695)
 - [core] Adapt `start:x11:dev` script
+- [weather/yr] The Yr weather provider won't start with a `updateInterval` below 600000 ms (10 minutes) in order to meet the terms of service.
 
 ### Removed
 

modules/default/weather/providers/yr.js

@@ -23,6 +23,10 @@ WeatherProvider.register("yr", {
 			Log.error("The Yr weather provider requires local storage.");
 			throw new Error("Local storage not available");
 		}
+		if (this.config.updateInterval < 600000) {
+			Log.error("The Yr weather provider requires an update interval of minimum 10 minutes (600 000ms).");
+			throw new Error("Update interval too short");
+		}
 		Log.info(`Weather provider: ${this.providerName} started.`);
 	},