Fix bug where cache is never checked

TextureHQ · Oct 7, 2024 · c023587 · c023587
1 parent 966dcb4
commit c023587
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "weather-plus",
-  "version": "0.0.17",
+  "version": "0.0.18",
   "description": "Weather Plus is a powerful wrapper around various Weather APIs that simplifies adding weather data to your application",
   "main": "./dist/cjs/index.js",
   "module": "./dist/esm/index.js",

diff --git a/src/weatherService.ts b/src/weatherService.ts
@@ -50,8 +50,13 @@ export class WeatherService {
     log(`Getting weather for (${lat}, ${lng})`);
     const geohash = getGeohash(lat, lng, 6);
 
-    const weather = await this.providers[this.provider].getWeather(lat, lng);
-    await this.cache.set(geohash, JSON.stringify(weather), 300); // Cache for 5 mins
-    return weather;
+    const cachedWeather = await this.cache.get(geohash);
+    if (cachedWeather) {
+      return JSON.parse(cachedWeather);
+    } else {
+      const weather = await this.providers[this.provider].getWeather(lat, lng);
+      await this.cache.set(geohash, JSON.stringify(weather), 300); // Cache for 5 mins
+      return weather;
+    }
   }
 }