Merge pull request #182 from tahnik/notif_fix

Notif fix
tahnik · Aug 27, 2017 · 28806ef · 28806ef
2 parents 218e7ef + f1d96dc
commit 28806ef
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 10 deletions.
diff --git a/app/src/main/js/actions/notifs.js b/app/src/main/js/actions/notifs.js
@@ -31,20 +31,22 @@ const fetchNotifs = () => (dispatch, getState) => {
     fetching = false;
     let noChange = true;
 
-    if (prevNotifs.num_unread === res.data.num_unread) {
+    if (prevNotifs && prevNotifs.num_unread === res.data.num_unread) {
       const nextnotifItems = res.data.items;
       let j = 0;
-      while (j < nextnotifItems.length) {
-        const prevItem = prevNotifs.items[j];
-        const nextItem = nextnotifItems[j];
-        if (
+      if (nextnotifItems) {
+        while (j < nextnotifItems.length) {
+          const prevItem = prevNotifs.items[j];
+          const nextItem = nextnotifItems[j];
+          if (
         prevItem.rant_id !== nextItem.rant_id
         || prevItem.read !== nextItem.read
       ) {
-          noChange = false;
-          break;
+            noChange = false;
+            break;
+          }
+          j += 1;
         }
-        j += 1;
       }
     } else {
       noChange = false;

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "devrantron",
-  "version": "1.3.3",
+  "version": "1.3.4",
   "main": "./app/build/app.js",
   "description": "An open source cross platform desktop application for devRant",
   "scripts": {
@@ -55,6 +55,9 @@
         "AppImage"
       ]
     },
+    "mac": {
+      "category": "public.app-category.developer-tools"
+    },
     "directories": {
       "output": "release"
     }