Skip to content

Commit 23ee155

Browse files
authored
Fixed wind speed units for Open-Meteo Weather Provider (#3054)
Resolved technical debt for [2964](#2964): - Set wind speed unit to m/s - Rename parameter `past_days` to `pastDays` to be consistent with all configs
1 parent 1b2785c commit 23ee155

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ _This release is scheduled to be released on 2023-04-01._
3838
- Cleanup jest coverage for patches
3939
- Update `stylelint` dependencies, switch to `stylelint-config-standard` and handle `stylelint` issues
4040
- Convert lots of callbacks to async/await
41+
- Fixed Open-Meteo wind speed units
4142

4243
### Fixed
4344

modules/default/weather/providers/openmeteo.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ WeatherProvider.register("openmeteo", {
2424
apiBase: OPEN_METEO_BASE,
2525
lat: 0,
2626
lon: 0,
27-
past_days: 0,
27+
pastDays: 0,
2828
type: "current"
2929
},
3030

@@ -227,12 +227,12 @@ WeatherProvider.register("openmeteo", {
227227
longitude: this.config.lon,
228228
timeformat: "unixtime",
229229
timezone: "auto",
230-
past_days: this.config.past_days ?? 0,
230+
past_days: this.config.pastDays ?? 0,
231231
daily: this.dailyParams,
232232
hourly: this.hourlyParams,
233233
// Fixed units as metric
234234
temperature_unit: "celsius",
235-
windspeed_unit: "kmh",
235+
windspeed_unit: "ms",
236236
precipitation_unit: "mm"
237237
};
238238

0 commit comments

Comments
 (0)