From 25ddf2c65178e2b8afe0709b48d5d30054208c04 Mon Sep 17 00:00:00 2001 From: Phil Oakley Date: Mon, 5 Sep 2016 19:06:21 -0600 Subject: [PATCH] currentweather module api url bugfix Running this on Raspberry Pi 2, node version v6.5.0, magicmirror@2.0.0 The current url created has a typo that causes the module to only display default weather units (Kelvin). After checking the website (http://openweathermap.org/current), there is no "/" at this location in the url. Removing it fixes the issue and allows the user to see the specified units in config.js. --- modules/default/currentweather/currentweather.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/default/currentweather/currentweather.js b/modules/default/currentweather/currentweather.js index db3d5de2..5265b099 100644 --- a/modules/default/currentweather/currentweather.js +++ b/modules/default/currentweather/currentweather.js @@ -164,7 +164,7 @@ Module.register("currentweather",{ * Calls processWeather on succesfull response. */ updateWeather: function() { - var url = this.config.apiBase + this.config.apiVersion + "/" + this.config.weatherEndpoint + '/' + this.getParams(); + var url = this.config.apiBase + this.config.apiVersion + "/" + this.config.weatherEndpoint + this.getParams(); var self = this; var retry = true;