Native Notifications

[manosim]

Review after #56 .

[JakeSidSmith]

This keeps the functions more pure, and I believe correctly picks the latest notification. I think the existing code would get the last of the new notifications.

// Check if notification is already in the store.
var newNotifications = _.filter(response, function (obj) {
  return !_.contains(self._previousNotifications, obj.id);
});

// Play Sound / Show Notification.
if (newNotifications && newNotifications.length) {
  if (playSound) {
    self.playSound();
  }
  if (showNotifications) {
    this.showNotification(countNew, response, {
      full_name: newNotifications[0].repository.full_name,
      subject: newNotifications[0].subject.title
    });
  }
}

[JakeSidSmith]

🚀