Merge pull request #170 from tahnik/hotfix3

Fixed update breaking notif
tahnik · Aug 17, 2017 · 218e7ef · 218e7ef
2 parents fb585da + 0057ad2
commit 218e7ef
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 16 deletions.
diff --git a/app/src/main/js/actions/notifs.js b/app/src/main/js/actions/notifs.js
@@ -31,27 +31,26 @@ const fetchNotifs = () => (dispatch, getState) => {
     fetching = false;
     let noChange = true;
 
-    // if (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 (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 (
         prevItem.rant_id !== nextItem.rant_id
         || prevItem.read !== nextItem.read
       ) {
-        noChange = false;
-        break;
+          noChange = false;
+          break;
+        }
+        j += 1;
       }
-      j += 1;
+    } else {
+      noChange = false;
     }
-    // } else {
-    //   noChange = false;
-    // }
 
     if (noChange) {
-      console.log('returning');
       return;
     }
     const notifs = {

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.2",
+  "version": "1.3.3",
   "main": "./app/build/app.js",
   "description": "An open source cross platform desktop application for devRant",
   "scripts": {