mumblebaj / MMM-NewsAPI

MagicMirror² module - displaying news articles from Newsapi.org API V2
MIT License
7 stars 5 forks source link

small typo which breaks excludeDomains #3

Closed POIMann closed 2 years ago

POIMann commented 2 years ago
diff --git a/node_helper.js b/node_helper.js
index 49cfe8e..00e2d3f 100644
--- a/node_helper.js
+++ b/node_helper.js
@@ -76,7 +76,7 @@ module.exports = NodeHelper.create({
             qs = Object.assign({}, qs, {"domains":d})
             }
             if (q.hasOwnProperty("excludeDomains") && q.excludeDomains !== "") {
-                var ed = q["excludeDoomains"].replace(/\s/g, "")
+                var ed = q["excludeDomains"].replace(/\s/g, "")
                 qs = Object.assign({}, qs, {"excludeDomains":ed})
             }
             if (q.hasOwnProperty("sources") && q.sources !== "") {
@@ -134,4 +134,4 @@ module.exports = NodeHelper.create({
             this.deconEverything(payload)
         }
     },
-})
\ No newline at end of file
+})
mumblebaj commented 2 years ago

Thanks for picking that up. It worked fine in testing. I guess I must have slipped when I was in the process of checking in my code. :-)

mumblebaj commented 2 years ago

Have pushed the fix. Thanks again for that.