From 34c3b627a3c2d40d09f082a9bdc8c1c3a7c6df2c Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 00:04:37 +0100
Subject: [PATCH 01/12] Notifications functionality

---
 src/js/stores/notifications.js | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/js/stores/notifications.js b/src/js/stores/notifications.js
index e3648a14f..540abf54a 100644
--- a/src/js/stores/notifications.js
+++ b/src/js/stores/notifications.js
@@ -25,23 +25,35 @@ var NotificationsStore = Reflux.createStore({
   isNewNotification: function (response) {
     var self = this;
     var playSound = SettingsStore.getSettings().playSound;
+    var showNotification = SettingsStore.getSettings().showNotification;
 
-    if (!playSound) { return; }
+    if (!playSound && !showNotification) { return; }
 
     // Check if notification is already in the store.
-    var isNew = false;
+    var countNew = 0;
     _.map(response, function (obj) {
       if (!_.contains(self._previousNotifications, obj.id)) {
-        isNew = true;
+        countNew ++;
       }
     });
 
     // Play Sound.
-    if (isNew) {
+    if (countNew > 0) {
       if (playSound) {
         var audio = new Audio('sounds/digi.wav');
         audio.play();
       }
+      if (showNotification) {
+        var body = (countNew = 1 ?
+          'You\'ve got a new notification' :
+          'You\'ve got ' + countNew + 'notifications'); 
+        var nativeNotification = new Notification('Gitify', {
+          body: body
+        });
+        nativeNotification.onclick = function () {
+          ipc.sendChannel('reopen-window');
+        };
+      }
     }
 
     // Now Reset the previousNotifications array.

From e6c58114bfb19156e36225d59e5472098f918c0b Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 00:07:05 +0100
Subject: [PATCH 02/12] Fix tests

---
 src/js/stores/notifications.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/js/stores/notifications.js b/src/js/stores/notifications.js
index 540abf54a..80dab5e43 100644
--- a/src/js/stores/notifications.js
+++ b/src/js/stores/notifications.js
@@ -46,7 +46,7 @@ var NotificationsStore = Reflux.createStore({
       if (showNotification) {
         var body = (countNew = 1 ?
           'You\'ve got a new notification' :
-          'You\'ve got ' + countNew + 'notifications'); 
+          'You\'ve got ' + countNew + 'notifications');
         var nativeNotification = new Notification('Gitify', {
           body: body
         });

From 17775b2959f19ac3a50618004e1bee1be6508823 Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 00:20:48 +0100
Subject: [PATCH 03/12] Show Notifications settings

---
 src/js/components/settings.js  | 11 ++++++++++-
 src/js/stores/notifications.js | 10 +++++-----
 src/js/stores/settings.js      | 16 ++++++++++++++--
 3 files changed, 29 insertions(+), 8 deletions(-)

diff --git a/src/js/components/settings.js b/src/js/components/settings.js
index bba22e1ab..8d98f0e3b 100644
--- a/src/js/components/settings.js
+++ b/src/js/components/settings.js
@@ -11,7 +11,8 @@ var SettingsPage = React.createClass({
     var settings = SettingsStore.getSettings();
     return {
       participating: settings.participating,
-      playSound: settings.playSound
+      playSound: settings.playSound,
+      showNotifications: settings.showNotifications
     };
   },
 
@@ -42,6 +43,14 @@ var SettingsPage = React.createClass({
               onChange={this.toggleSetting.bind(this, 'playSound')} />
           </div>
         </div>
+        <div className='row'>
+          <div className='col-xs-8'>Show notifications</div>
+          <div className='col-xs-4'>
+            <Toggle
+              defaultChecked={this.state.showNotifications}
+              onChange={this.toggleSetting.bind(this, 'showNotifications')} />
+          </div>
+        </div>
         <div className='row'>
           <button
             className='btn btn-block btn-danger btn-close'
diff --git a/src/js/stores/notifications.js b/src/js/stores/notifications.js
index 80dab5e43..8d50e0624 100644
--- a/src/js/stores/notifications.js
+++ b/src/js/stores/notifications.js
@@ -25,9 +25,9 @@ var NotificationsStore = Reflux.createStore({
   isNewNotification: function (response) {
     var self = this;
     var playSound = SettingsStore.getSettings().playSound;
-    var showNotification = SettingsStore.getSettings().showNotification;
+    var showNotifications = SettingsStore.getSettings().showNotifications;
 
-    if (!playSound && !showNotification) { return; }
+    if (!playSound && !showNotifications) { return; }
 
     // Check if notification is already in the store.
     var countNew = 0;
@@ -43,10 +43,10 @@ var NotificationsStore = Reflux.createStore({
         var audio = new Audio('sounds/digi.wav');
         audio.play();
       }
-      if (showNotification) {
-        var body = (countNew = 1 ?
+      if (showNotifications) {
+        var body = (countNew == 1 ?
           'You\'ve got a new notification' :
-          'You\'ve got ' + countNew + 'notifications');
+          'You\'ve got ' + countNew + ' notifications.');
         var nativeNotification = new Notification('Gitify', {
           body: body
         });
diff --git a/src/js/stores/settings.js b/src/js/stores/settings.js
index a4b71983d..9965ad63d 100644
--- a/src/js/stores/settings.js
+++ b/src/js/stores/settings.js
@@ -9,8 +9,9 @@ var SettingsStore = Reflux.createStore({
 
     if (!settings) {
       settings = {
-        'participating': false,
-        'playSound': true
+        participating: false,
+        playSound: true,
+        showNotifications: true
       };
     }
 
@@ -18,7 +19,18 @@ var SettingsStore = Reflux.createStore({
       settings = JSON.parse(settings);
     }
 
+    if (!settings.participating) {
+      settings.participating = false;
+    }
+    if (!settings.playSound) {
+      settings.playSound = true;
+    }
+    if (!settings.showNotifications) {
+      settings.showNotifications = true;
+    }
+
     this._settings = settings;
+    window.localStorage.setItem('settings', JSON.stringify(this._settings));
   },
 
   getSettings: function () {

From 9056d3830bfab7352832714a5d1b853976d1ea78 Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 00:35:31 +0100
Subject: [PATCH 04/12] Fix tests

---
 src/js/__tests__/components/notification.js  |  3 +++
 src/js/__tests__/components/notifications.js |  3 +++
 src/js/__tests__/components/settings.js      |  3 +++
 src/js/__tests__/stores/notifications.js     | 13 +++++++++++--
 4 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/src/js/__tests__/components/notification.js b/src/js/__tests__/components/notification.js
index a789dcf4c..8d6ae2b32 100644
--- a/src/js/__tests__/components/notification.js
+++ b/src/js/__tests__/components/notification.js
@@ -34,6 +34,9 @@ describe('Test for Notification Component', function () {
       item: false,
       getItem: function () {
         return this.item;
+      },
+      setItem: function (item) {
+        this.item = item;
       }
     };
 
diff --git a/src/js/__tests__/components/notifications.js b/src/js/__tests__/components/notifications.js
index 7b69d6463..abc5c59ff 100644
--- a/src/js/__tests__/components/notifications.js
+++ b/src/js/__tests__/components/notifications.js
@@ -34,6 +34,9 @@ describe('Test for Notifications Component', function () {
       item: false,
       getItem: function () {
         return this.item;
+      },
+      setItem: function (item) {
+        this.item = item;
       }
     };
 
diff --git a/src/js/__tests__/components/settings.js b/src/js/__tests__/components/settings.js
index da9177a7b..dd334f8ef 100644
--- a/src/js/__tests__/components/settings.js
+++ b/src/js/__tests__/components/settings.js
@@ -28,6 +28,9 @@ describe('Test for Settings Component', function () {
       item: false,
       getItem: function () {
         return this.item;
+      },
+      setItem: function (item) {
+        this.item = item;
       }
     };
 
diff --git a/src/js/__tests__/stores/notifications.js b/src/js/__tests__/stores/notifications.js
index 53fc68c15..25d51ac3b 100644
--- a/src/js/__tests__/stores/notifications.js
+++ b/src/js/__tests__/stores/notifications.js
@@ -27,17 +27,26 @@ describe('Tests for NotificationsStore', function () {
       item: false,
       getItem: function () {
         return this.item;
+      },
+      setItem: function (item) {
+        this.item = item;
       }
     };
 
     // Mock Audio
-    window.Audio = function (src) {
-      console.log('Loading Audio: ' + src);
+    window.Audio = function () {
       return {
         play: function () {}
       };
     };
 
+    // Mock Notifications
+    window.Notification = function () {
+      return {
+        onClick: function () {}
+      };
+    };
+
     Actions = require('../../actions/actions.js');
     apiRequests = require('../../utils/api-requests.js');
     NotificationsStore = require('../../stores/notifications.js');

From c15c6626956a857afacb846ff04bfc7f8d2fa33e Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 01:32:25 +0100
Subject: [PATCH 05/12] Show more info if one 1 new notification

---
 src/js/stores/notifications.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/js/stores/notifications.js b/src/js/stores/notifications.js
index 8d50e0624..130520fa8 100644
--- a/src/js/stores/notifications.js
+++ b/src/js/stores/notifications.js
@@ -44,10 +44,13 @@ var NotificationsStore = Reflux.createStore({
         audio.play();
       }
       if (showNotifications) {
+        var title = (countNew == 1 ?
+          'Gitify - ' + response[0].repository.full_name :
+          'Gitify');
         var body = (countNew == 1 ?
-          'You\'ve got a new notification' :
+          response[0].subject.title :
           'You\'ve got ' + countNew + ' notifications.');
-        var nativeNotification = new Notification('Gitify', {
+        var nativeNotification = new Notification(title, {
           body: body
         });
         nativeNotification.onclick = function () {

From 06093ae12d633c405166954e815071c8b212ab19 Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 19:40:32 +0100
Subject: [PATCH 06/12] Split to 'sound-notification' store & get the correct
 full_name & subject title

---
 src/js/actions/actions.js           |  1 +
 src/js/stores/notifications.js      | 47 +-----------------
 src/js/stores/sound-notification.js | 74 +++++++++++++++++++++++++++++
 3 files changed, 76 insertions(+), 46 deletions(-)
 create mode 100644 src/js/stores/sound-notification.js

diff --git a/src/js/actions/actions.js b/src/js/actions/actions.js
index d26dab6eb..6c476429e 100644
--- a/src/js/actions/actions.js
+++ b/src/js/actions/actions.js
@@ -5,6 +5,7 @@ var Actions = Reflux.createActions({
   'login': {},
   'logout': {},
   'getNotifications': {asyncResult: true},
+  'isNewNotification': {},
   'updateSearchTerm': {},
   'clearSearchTerm': {},
   'setSetting': {}
diff --git a/src/js/stores/notifications.js b/src/js/stores/notifications.js
index 130520fa8..793fbf3ce 100644
--- a/src/js/stores/notifications.js
+++ b/src/js/stores/notifications.js
@@ -11,7 +11,6 @@ var NotificationsStore = Reflux.createStore({
 
   init: function () {
     this._notifications = [];
-    this._previousNotifications = [];
   },
 
   updateTrayIcon: function (notifications) {
@@ -22,50 +21,6 @@ var NotificationsStore = Reflux.createStore({
     }
   },
 
-  isNewNotification: function (response) {
-    var self = this;
-    var playSound = SettingsStore.getSettings().playSound;
-    var showNotifications = SettingsStore.getSettings().showNotifications;
-
-    if (!playSound && !showNotifications) { return; }
-
-    // Check if notification is already in the store.
-    var countNew = 0;
-    _.map(response, function (obj) {
-      if (!_.contains(self._previousNotifications, obj.id)) {
-        countNew ++;
-      }
-    });
-
-    // Play Sound.
-    if (countNew > 0) {
-      if (playSound) {
-        var audio = new Audio('sounds/digi.wav');
-        audio.play();
-      }
-      if (showNotifications) {
-        var title = (countNew == 1 ?
-          'Gitify - ' + response[0].repository.full_name :
-          'Gitify');
-        var body = (countNew == 1 ?
-          response[0].subject.title :
-          'You\'ve got ' + countNew + ' notifications.');
-        var nativeNotification = new Notification(title, {
-          body: body
-        });
-        nativeNotification.onclick = function () {
-          ipc.sendChannel('reopen-window');
-        };
-      }
-    }
-
-    // Now Reset the previousNotifications array.
-    self._previousNotifications = [];
-    _.map(response, function (obj) {
-      self._previousNotifications.push(obj.id);
-    });
-  },
-
   onGetNotifications: function () {
     var self = this;
     var participating = SettingsStore.getSettings().participating;
@@ -78,7 +33,7 @@ var NotificationsStore = Reflux.createStore({
           // Success - Do Something.
           Actions.getNotifications.completed(response.body);
           self.updateTrayIcon(response.body);
-          self.isNewNotification(response.body);
+          Actions.isNewNotification(response.body);
         } else {
           // Error - Show messages.
           Actions.getNotifications.failed(err);
diff --git a/src/js/stores/sound-notification.js b/src/js/stores/sound-notification.js
new file mode 100644
index 000000000..85d34f0d7
--- /dev/null
+++ b/src/js/stores/sound-notification.js
@@ -0,0 +1,74 @@
+var ipc = window.require('ipc');
+var Reflux = require('reflux');
+var _ = require('underscore');
+
+var Actions = require('../actions/actions');
+var SettingsStore = require('../settings');
+
+var SoundNotificationStore = Reflux.createStore({
+  listenables: Actions,
+
+  init: function () {
+    this._previousNotifications = [];
+  },
+
+  playSound: function () {
+    var audio = new Audio('sounds/digi.wav');
+    audio.play();
+  },
+
+  showNotification: function (countNew, response, latestNotification) {
+    var title = (countNew == 1 ?
+      'Gitify - ' + latestNotification.full_name :
+      'Gitify');
+    var body = (countNew == 1 ?
+      latestNotification.subject :
+      'You\'ve got ' + countNew + ' notifications.');
+    var nativeNotification = new Notification(title, {
+      body: body
+    });
+    nativeNotification.onclick = function () {
+      ipc.sendChannel('reopen-window');
+    };
+  },
+
+  onIsNewNotification: function (response) {
+    var self = this;
+    var playSound = SettingsStore.getSettings().playSound;
+    var showNotifications = SettingsStore.getSettings().showNotifications;
+
+    if (!playSound && !showNotifications) { return; }
+
+    // Check if notification is already in the store.
+    var countNew = 0;
+    var latestNotification = {};
+    _.map(response, function (obj) {
+      if (!_.contains(self._previousNotifications, obj.id)) {
+        countNew ++;
+        latestNotification = {
+          full_name: obj.repository.full_name,
+          subject: obj.subject.title
+        };
+      }
+    });
+
+    // Play Sound / Show Notification.
+    if (countNew > 0) {
+      if (playSound) {
+        self.playSound();
+      }
+      if (showNotifications) {
+        self.showNotification(countNew, response, latestNotification);
+      }
+    }
+
+    // Now Reset the previousNotifications array.
+    self._previousNotifications = [];
+    _.map(response, function (obj) {
+      self._previousNotifications.push(obj.id);
+    });
+  }
+
+});
+
+module.exports = SoundNotificationStore;

From 62532c96ace40401247e39b99a23a8ffb719e16f Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 19:49:17 +0100
Subject: [PATCH 07/12] Tests for 'sound-notification' store

---
 package.json                                  |  3 +-
 src/js/__tests__/stores/notifications.js      | 14 ----
 src/js/__tests__/stores/sound-notification.js | 77 +++++++++++++++++++
 src/js/stores/sound-notification.js           |  2 +-
 4 files changed, 80 insertions(+), 16 deletions(-)
 create mode 100644 src/js/__tests__/stores/sound-notification.js

diff --git a/package.json b/package.json
index 334fd0eb7..63c70cf7e 100644
--- a/package.json
+++ b/package.json
@@ -76,7 +76,8 @@
       "src/js/stores/auth.js": true,
       "src/js/stores/notifications.js": true,
       "src/js/stores/search.js": true,
-      "src/js/stores/settings.js": true
+      "src/js/stores/settings.js": true,
+      "src/js/stores/sound-notification.js": true
     },
     "unmockedModulePathPatterns": [
       "node_modules/react",
diff --git a/src/js/__tests__/stores/notifications.js b/src/js/__tests__/stores/notifications.js
index 25d51ac3b..55f3631a1 100644
--- a/src/js/__tests__/stores/notifications.js
+++ b/src/js/__tests__/stores/notifications.js
@@ -33,20 +33,6 @@ describe('Tests for NotificationsStore', function () {
       }
     };
 
-    // Mock Audio
-    window.Audio = function () {
-      return {
-        play: function () {}
-      };
-    };
-
-    // Mock Notifications
-    window.Notification = function () {
-      return {
-        onClick: function () {}
-      };
-    };
-
     Actions = require('../../actions/actions.js');
     apiRequests = require('../../utils/api-requests.js');
     NotificationsStore = require('../../stores/notifications.js');
diff --git a/src/js/__tests__/stores/sound-notification.js b/src/js/__tests__/stores/sound-notification.js
new file mode 100644
index 000000000..cedc6d5cc
--- /dev/null
+++ b/src/js/__tests__/stores/sound-notification.js
@@ -0,0 +1,77 @@
+/*global jest, describe, it, expect, spyOn, beforeEach */
+
+'use strict';
+
+jest.dontMock('reflux');
+jest.dontMock('../../stores/sound-notification.js');
+jest.dontMock('../../utils/api-requests.js');
+jest.dontMock('../../actions/actions.js');
+
+describe('Tests for SoundNotificationStore', function () {
+
+  var SoundNotificationStore, Actions;
+
+  beforeEach(function () {
+
+    // Mock Electron's window.require
+    window.require = function () {
+      return {
+        sendChannel: function () {
+          return;
+        }
+      };
+    };
+
+    // Mock localStorage
+    window.localStorage = {
+      item: false,
+      getItem: function () {
+        return this.item;
+      },
+      setItem: function (item) {
+        this.item = item;
+      }
+    };
+
+    // Mock Audio
+    window.Audio = function () {
+      return {
+        play: function () {}
+      };
+    };
+
+    // Mock Notifications
+    window.Notification = function () {
+      return {
+        onClick: function () {}
+      };
+    };
+
+    Actions = require('../../actions/actions.js');
+    SoundNotificationStore = require('../../stores/sound-notification.js');
+  });
+
+  it('should get a payload and check if it should play sound & show notification.', function () {
+
+    spyOn(SoundNotificationStore, 'showNotification');
+
+    var payload = [{
+      'id': '1',
+      'repository': {
+        'id': 1296269,
+        'full_name': 'octocat/Hello-World',
+        'description': 'This your first repo!'
+      },
+      'subject': {
+        'title': 'Greetings',
+        'url': 'https://api.github.com/repos/octokit/octokit.rb/issues/123'
+      }
+    }];
+
+    SoundNotificationStore.onIsNewNotification(payload);
+
+    expect(SoundNotificationStore.showNotification).toHaveBeenCalled();
+
+  });
+
+});
diff --git a/src/js/stores/sound-notification.js b/src/js/stores/sound-notification.js
index 85d34f0d7..e77b85373 100644
--- a/src/js/stores/sound-notification.js
+++ b/src/js/stores/sound-notification.js
@@ -3,7 +3,7 @@ var Reflux = require('reflux');
 var _ = require('underscore');
 
 var Actions = require('../actions/actions');
-var SettingsStore = require('../settings');
+var SettingsStore = require('../stores/settings');
 
 var SoundNotificationStore = Reflux.createStore({
   listenables: Actions,

From b84c90928e97d99ef107c32856409233be58b2ce Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 20:34:10 +0100
Subject: [PATCH 08/12] Require 'sound-notification' store to be included in
 the build

---
 src/js/stores/notifications.js      | 2 ++
 src/js/stores/sound-notification.js | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/js/stores/notifications.js b/src/js/stores/notifications.js
index 793fbf3ce..7ecc3ebd7 100644
--- a/src/js/stores/notifications.js
+++ b/src/js/stores/notifications.js
@@ -6,6 +6,8 @@ var Actions = require('../actions/actions');
 var apiRequests = require('../utils/api-requests');
 var SettingsStore = require('../stores/settings');
 
+require('../stores/sound-notification');
+
 var NotificationsStore = Reflux.createStore({
   listenables: Actions,
 
diff --git a/src/js/stores/sound-notification.js b/src/js/stores/sound-notification.js
index e77b85373..458543b70 100644
--- a/src/js/stores/sound-notification.js
+++ b/src/js/stores/sound-notification.js
@@ -58,7 +58,7 @@ var SoundNotificationStore = Reflux.createStore({
         self.playSound();
       }
       if (showNotifications) {
-        self.showNotification(countNew, response, latestNotification);
+        this.showNotification(countNew, response, latestNotification);
       }
     }
 

From eef99c8d9e1e36fb72cde64abf1a79b49c273eb8 Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 20:46:40 +0100
Subject: [PATCH 09/12] Mocks for Audio & notifcations

---
 src/js/__tests__/stores/notifications.js | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/src/js/__tests__/stores/notifications.js b/src/js/__tests__/stores/notifications.js
index 55f3631a1..25d51ac3b 100644
--- a/src/js/__tests__/stores/notifications.js
+++ b/src/js/__tests__/stores/notifications.js
@@ -33,6 +33,20 @@ describe('Tests for NotificationsStore', function () {
       }
     };
 
+    // Mock Audio
+    window.Audio = function () {
+      return {
+        play: function () {}
+      };
+    };
+
+    // Mock Notifications
+    window.Notification = function () {
+      return {
+        onClick: function () {}
+      };
+    };
+
     Actions = require('../../actions/actions.js');
     apiRequests = require('../../utils/api-requests.js');
     NotificationsStore = require('../../stores/notifications.js');

From c2b47ce0402be7eef28d69da5da6b525a5773ab5 Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 13 Jun 2015 22:09:33 +0100
Subject: [PATCH 10/12] Remove coveralls!

---
 package.json | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/package.json b/package.json
index 63c70cf7e..c63b53c69 100644
--- a/package.json
+++ b/package.json
@@ -10,8 +10,7 @@
     "watch": "grunt build && npm build && npm run watch-js | grunt watch",
     "start": "electron .",
     "dist": "rm -rf Gitify.app/ && electron-packager . Gitify --platform=darwin --arch=x64 --version=0.27.2 --icon=images/app-icon.icns --prune --ignore=src",
-    "test": "jsxhint --reporter node_modules/jshint-stylish/stylish.js 'src/**/*.js', 'index.js' --exclude 'Gruntfile.js' && jscs 'src/js/' && jest",
-    "coveralls": "cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js"
+    "test": "jsxhint --reporter node_modules/jshint-stylish/stylish.js 'src/**/*.js', 'index.js' --exclude 'Gruntfile.js' && jscs 'src/js/' && jest"
   },
   "jshintConfig": {
     "browserify": true,

From adabd091717b7a18b17d306e0cf0ef35c1d4fd1d Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sun, 14 Jun 2015 19:14:56 +0100
Subject: [PATCH 11/12] Jake's comments

---
 src/js/stores/sound-notification.js | 28 +++++++++++-----------------
 1 file changed, 11 insertions(+), 17 deletions(-)

diff --git a/src/js/stores/sound-notification.js b/src/js/stores/sound-notification.js
index 458543b70..271fe80e2 100644
--- a/src/js/stores/sound-notification.js
+++ b/src/js/stores/sound-notification.js
@@ -10,11 +10,11 @@ var SoundNotificationStore = Reflux.createStore({
 
   init: function () {
     this._previousNotifications = [];
+    this._audio = new Audio('sounds/digi.wav');
   },
 
   playSound: function () {
-    var audio = new Audio('sounds/digi.wav');
-    audio.play();
+    this._audio.play();
   },
 
   showNotification: function (countNew, response, latestNotification) {
@@ -40,32 +40,26 @@ var SoundNotificationStore = Reflux.createStore({
     if (!playSound && !showNotifications) { return; }
 
     // Check if notification is already in the store.
-    var countNew = 0;
-    var latestNotification = {};
-    _.map(response, function (obj) {
-      if (!_.contains(self._previousNotifications, obj.id)) {
-        countNew ++;
-        latestNotification = {
-          full_name: obj.repository.full_name,
-          subject: obj.subject.title
-        };
-      }
+    var newNotifications = _.filter(response, function (obj) {
+      return !_.contains(self._previousNotifications, obj.id);
     });
 
     // Play Sound / Show Notification.
-    if (countNew > 0) {
+    if (newNotifications && newNotifications.length) {
       if (playSound) {
         self.playSound();
       }
       if (showNotifications) {
-        this.showNotification(countNew, response, latestNotification);
+        this.showNotification(newNotifications.length, response, {
+          full_name: newNotifications[0].repository.full_name,
+          subject: newNotifications[0].subject.title
+        });
       }
     }
 
     // Now Reset the previousNotifications array.
-    self._previousNotifications = [];
-    _.map(response, function (obj) {
-      self._previousNotifications.push(obj.id);
+    self._previousNotifications = _.map(response, function (obj) {
+      return obj.id;
     });
   }
 

From eca42dd3d3f75bd6c3b53e642c534ae2304fabd0 Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sun, 14 Jun 2015 19:29:56 +0100
Subject: [PATCH 12/12] Revert setting audio in store's init

---
 src/js/stores/sound-notification.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/js/stores/sound-notification.js b/src/js/stores/sound-notification.js
index 271fe80e2..af37a6f11 100644
--- a/src/js/stores/sound-notification.js
+++ b/src/js/stores/sound-notification.js
@@ -10,11 +10,11 @@ var SoundNotificationStore = Reflux.createStore({
 
   init: function () {
     this._previousNotifications = [];
-    this._audio = new Audio('sounds/digi.wav');
   },
 
   playSound: function () {
-    this._audio.play();
+    var audio = new Audio('sounds/digi.wav');
+    audio.play();
   },
 
   showNotification: function (countNew, response, latestNotification) {