Exclude devices without monitoring API e.g hs100

Fixes #7
jamesbarnett91 · Nov 18, 2018 · 01f4993 · 01f4993
1 parent 27f8862
commit 01f4993
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/services/device-manager.js b/services/device-manager.js
@@ -7,12 +7,19 @@ var devices = [];
 client.startDiscovery({
     deviceTypes: ['plug'],
     discoveryTimeout: 20000
-  }).on('plug-new', plug => {
-  console.log('Found device: ' + plug.alias + ' [' + plug.deviceId + ']');
-  devices.push(plug);
+  }).on('plug-new', registerPlug);
 
-  dataLogger.startLogging(plug);
-});
+function registerPlug(plug) {
+
+  if (plug.supportsEmeter) {
+    console.log('Found device with energy monitor support: ' + plug.alias + ' [' + plug.deviceId + ']');
+    devices.push(plug);
+    dataLogger.startLogging(plug);
+  } else {
+    console.log('Skipping device: ' + plug.alias + ' [' + plug.deviceId + ']. Energy monitoring not supported.');
+  }
+
+}
 
 module.exports.getDevice = function(deviceId) {