Fixed issue where push notifications would get sent out for narmal st…

…ates when the node server starts up
sbender9 · Jan 5, 2017 · bcf8ba4 · bcf8ba4
1 parent 512256e
commit bcf8ba4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -35,7 +35,7 @@ module.exports = function(app) {
       context: "vessels.self",
       subscribe: [{
         path: "notifications.*",
-        minPeriod: 1000
+        policy: 'instant'
       }]
     }
 
@@ -192,13 +192,17 @@ function handleNotificationDelta(app, id, notification, last_states)
              && typeof value.value.message != 'undefined'
              && value.value.message != null )
         {
-          if ( last_states[value.path] == null
-               || last_states[value.path] != value.value.state )
+          if ( (last_states[value.path] == null
+                && value.value.state != 'normal')
+               || ( last_states[value.path] != null
+                    && last_states[value.path] != value.value.state) )
           {
             last_states[value.path] = value.value.state
             debug("message:" + value.value.message)
             send_push(app, device, value.value.message, value.path)
           }
+          else
+            debug("skipped")
         }
         else if ( last_states[value.path] )
         {

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "signalk-push-notifications",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "sognalk-node-server plugin that pushes SignalK notifications to Amazon SNS",
   "main": "index.js",
   "scripts": {