bt-sensors-plugin-sk 1.2.0-beta.0.0.7 → 1.2.0-beta.0.0.9

package/.vscode/launch.json

@@ -0,0 +1,7 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": []
+}

package/BTSensor.js

@@ -100,7 +100,6 @@ function preparePath(obj, str) {
  */
 
 class BTSensor extends EventEmitter {
-    //static metadata=new Map()
  
     static DEFAULTS = require('./plugin_defaults.json');
     
@@ -318,6 +317,7 @@ class BTSensor extends EventEmitter {
 
         //create the 'name' parameter
         this.addDefaultParam("name")
+            .default=this?.currentProperties?.Name
 
         //create the 'location' parameter
 
@@ -365,13 +365,15 @@ class BTSensor extends EventEmitter {
             }
             else 
                 await this.initGATTNotifications()
+        }).catch((e)=>{
+            this.app.debug(`Unable to activate GATT connection for ${this.getName()} (${this.getMacAddress()}): ${e}`)
         })
     }
 
     /**
      * Add a metadatum instance to the sensor instance
      *  
-     * @param {String} tag 
+     * @param {String} tag
      * @param  {...any} args 
      * @returns {this.Metadatum} the new metadatum instance
      */
@@ -552,6 +554,9 @@ class BTSensor extends EventEmitter {
     getParams(){
         return this._schema.properties.params.properties
     }
+    getParameter(param){
+        return this.getParams()[param]
+    }
     getGATTParams(){
         return this._schema.properties.gattParams.properties
        

package/README.md

@@ -2,6 +2,16 @@
 
 ## What's New
 
+## 1.2.0-beta-0.0.9
+
+Fixes to config page RSSI update.
+
+## 1.2.0-beta-0.0.8
+
+Improved handling of changed state on config page.
+
+Several small fixes to various sensor classes.
+
 ## 1.2.0-beta-0.0.7
 
 Added zone config param for environmental sensors. Removed location parameter. 

package/index.js

@@ -21,15 +21,13 @@ class MissingSensor  {
 		this.addPath=BTSensor.prototype.addPath.bind(this)
 		this.addParameter=BTSensor.prototype.addParameter.bind(this)
 		this.addDefaultPath=BTSensor.prototype.addDefaultPath.bind(this)
-		this.addDefaultParam=BTSensor.prototype.addDefaultPath.bind(this)
-		this.addDefaultParam=BTSensor.prototype.addDefaultPath.bind(this)
+		this.addDefaultParam=BTSensor.prototype.addDefaultParam.bind(this)
 		this.getPath=BTSensor.prototype.getPath.bind(this)
 
 		this.getJSONSchema = BTSensor.prototype.getJSONSchema.bind(this)
 		this.initSchema = BTSensor.prototype.initSchema.bind(this)
 
 		this.initSchema()
-
 		var keys = Object.keys(config?.paths??{})
 
 		keys.forEach((key)=>{
@@ -321,13 +319,13 @@ module.exports =   function (app) {
 		}
 
 		function getSensorInfo(sensor){
-			if (sensor.getName()=="vent") 
-				debugger
+	
+			const etslc = sensor.elapsedTimeSinceLastContact()
 			return { mac: sensor.getMacAddress(),
 				     name: sensor.getName(),
 					 RSSI: sensor.getRSSI(),
 					 signalStrength: sensor.getSignalStrength(),
-					 lastContactDelta: sensor. elapsedTimeSinceLastContact()
+					 lastContactDelta: etslc
 			}
 		}
 
@@ -390,7 +388,6 @@ module.exports =   function (app) {
 			adapter.waitDevice(config.mac_address,(config?.discoveryTimeout??30)*1000)
 			.then(async (device)=> { 
 				if (startNumber != starts ) {
-					debugger
 					return
 				}
 				//app.debug(`Found ${config.mac_address}`)
@@ -400,8 +397,14 @@ module.exports =   function (app) {
 					reject ( `Device is blacklisted (${s.reasonForBlacklisting()}).`)
 				else{
 					addSensorToList(s)
+					s._lastRSSI=-1*Infinity
 					s.on("RSSI",(()=>{
-						updateSensor(s)
+						if (Date.now()-s._lastRSSI > 20000) { //only update RSSI on client every five seconds
+							//app.debug(`${s.getMacAddress()} ${Date.now()-s._lastRSSI}`) 
+							s._lastRSSI=Date.now()
+							updateSensor(s)
+						}	
+
 					}))
 					resolve(s)
 				}
@@ -443,7 +446,7 @@ module.exports =   function (app) {
 				app.setPluginError(msg)
 			}
 			//if we're here ain't got no class for the device
-			var sensor
+			var sensor 
 			if (config.params?.sensorClass){
 				var c = classMap.get(config.params.sensorClass)
 			} else{
@@ -636,15 +639,18 @@ module.exports =   function (app) {
 			}
 		}
 		const minTimeout=Math.min(...deviceConfigs.map((dc)=>dc?.discoveryTimeout??options.discoveryTimeout))
-
+		const intervalTimeout = ((minTimeout==Infinity)?(options?.discoveryTimeout??plugin.schema.properties.discoveryTimeout.default):minTimeout)*1000
 		deviceHealthID = setInterval( ()=> {
 			sensorMap.forEach((sensor)=>{
 				const config = getDeviceConfig(sensor.getMacAddress())
 				const dt = config?.discoveryTimeout??options.discoveryTimeout
-				if (sensor.elapsedTimeSinceLastContact()> dt)
+				const lc=sensor.elapsedTimeSinceLastContact()
+				if (lc > dt) { 
+					app.debug(`${sensor.getMacAddress()} not heard from in ${lc} seconds`)
 					channel.broadcast(getSensorInfo(sensor), "sensorchanged")
+				}
 			})
-		}, (minTimeout==Infinity)?(options?.discoveryTimeout??plugin.schema.properties.discoveryTimeout.default):minTimeout)
+		}, intervalTimeout)
 		
 		if (!options.hasOwnProperty("discoveryInterval" )) //no config -- first run
 			options.discoveryInterval = plugin.schema.properties.discoveryInterval.default

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bt-sensors-plugin-sk",
-  "version": "1.2.0-beta.0.0.7",
+  "version": "1.2.0-beta.0.0.9",
   "description": "Bluetooth Sensors for Signalk -- support for Victron devices, RuuviTag, Xiaomi, ATC and Inkbird, Ultrasonic wind meters, Mopeka tank readers, Renogy Battery and Solar Controllers, Aranet4 environment sensors, SwitchBot temp and humidity sensors, KilovaultHLXPlus smart batteries, and Govee GVH51xx temp sensors",
   "main": "index.js",
   "dependencies": {

package/plugin_defaults.json

@@ -22,6 +22,7 @@
     "environment":{
         "temperature": 
             {
+                "title":"Current zone's temperature",
                 "unit":"K",
                 "default": "environment.{zone}.temperature"
             },
@@ -32,15 +33,15 @@
             },
         "relativeHumidity":
             {
-                "title":"Current relative humidity",
+                "title":"Current zone's relative humidity",
                 "unit":"ratio",
-                "default":"environment.{zone}.humidity"
+                "default":"environment.{zone}.relativeHumidity"
             },
         "pressure":
             {
-                "title": "Current outside air ambient pressure",
+                "title": "Current zone's ambient air pressure",
                 "unit":"Pa",
-                "default":"environment.{zone}.humidity"
+                "default":"environment.{zone}.pressure"
             }
     },
     "electrical":{

package/public/893.js

@@ -1 +1 @@
-"use strict";(self.webpackChunkbt_sensors_plugin_sk=self.webpackChunkbt_sensors_plugin_sk||[]).push([[893],{2995:(e,t,n)=>{n.r(t),n.d(t,{default:()=>v});var a=n(3490),s=n(4810),o=n(4147),r=n.n(o),l=n(7606),c=n(4952),i=n(3768),u=n(1431),d=n(9676),m=n(7041),f=n(3657),g=n(5027),p=n(7265),h=n(6890),E=n(8207);const w=e=>console.log.bind(console,e);var S;const v=e=>{const[t,n]=(0,o.useState)({}),[v,b]=(0,o.useState)({}),[y,A]=(0,o.useState)({}),[$,k]=(0,o.useState)({"ui:submitButtonOptions":{props:{disabled:!1,className:"btn btn-info"},norender:!0,submitText:"Submit"}}),[x,D]=(0,o.useState)([]),[C,T]=(0,o.useState)(),[j,_]=(0,o.useState)(new Map),[O,N]=(0,o.useState)({progress:0,maxTimeout:100,deviceCount:0,totalDevices:0}),[M,J]=(0,o.useState)("unknown"),[L,U]=(0,o.useState)();function B(e,t){console.log(`sending ${e}`),console.log(t);const n=new Headers;return n.append("Content-Type","application/json"),fetch(`/plugins/bt-sensors-plugin-sk/${e}`,{credentials:"include",method:"POST",body:JSON.stringify(t),headers:n})}async function H(e){return console.log(`fetching ${e}`),fetch(`/plugins/bt-sensors-plugin-sk/${e}`,{credentials:"include"})}async function I(){const e=await H("progress");if(200!=e.status)throw new Error(`Unable get progres: ${e.statusText} (${e.status}) `);const t=await e.json();return console.log(t),t}function P(){console.log("refreshing sensor map"),async function(){const e=await H("sensors");if(200!=e.status)throw new Error(`Unable get sensor data: ${e.statusText} (${e.status}) `);const t=await e.json();return console.log(t),t}().then((e=>{_(new Map(e.map((e=>[e.info.mac,e]))))})).catch((e=>{U(e)}))}return(0,o.useEffect)((()=>{H("sendPluginState").then((async e=>{const t=await e.json();J(t.state),console.log("Setting up eventsource");const n=new EventSource("/plugins/bt-sensors-plugin-sk/sse");return n.addEventListener("newsensor",(e=>{let t=JSON.parse(e.data);S.has(t.info.mac)||(console.log(`New sensor: ${t.info.mac}`),_(new Map(S.set(t.info.mac,t))))})),n.addEventListener("sensorchanged",(e=>{let t=JSON.parse(e.data);if(S.has(t.mac)){let e=S.get(t.mac);Object.assign(e.info,t),_(new Map(S))}})),n.addEventListener("progress",(e=>{const t=JSON.parse(e.data);N(t),console.log(t)})),n.addEventListener("pluginstate",(e=>{const t=JSON.parse(e.data);J(t.state)})),()=>n.close()})).catch((e=>{U(e)}))}),[]),(0,o.useEffect)((()=>{"started"==M?(P(),async function(){const e=await H("base");if(200!=e.status)throw new Error(`Unable get base data: ${e.statusText} (${e.status}) `);const t=await e.json();return console.log(t),t}().then((e=>{n(e.schema),b(e.data)})).catch((e=>{U(e)})),I().then((e=>{N(e)})).catch((e=>{U(e)}))):(_(new Map),n({}),b({}))}),[M]),(0,o.useEffect)((()=>{console.log(L)}),[L]),(0,o.useEffect)((()=>{S=j,D(Array.from(j.entries()).map((e=>{const t=j.get(e[0]),n=t.config,a=Object.keys(n).length>0;return r().createElement(h.A,{action:!0,onClick:()=>{t&&(n.mac_address=e[0],A(t.schema),T(n))}},r().createElement("div",{class:"d-flex justify-content-between align-items-center",style:a?{fontWeight:"normal"}:{fontStyle:"italic"}},`${t.info.name} MAC: ${t.info.mac} RSSI: ${s=t.info.RSSI,null==s?NaN:s}`,r().createElement("div",{class:"d-flex justify-content-between "},function(e){return null==e.info.lastContactDelta||e.info.lastContactDelta>e.config.discoveryTimeout?r().createElement(i.A,null):e.info.signalStrength>80?r().createElement(u.A,null):e.info.signalStrength>60?r().createElement(d.A,null):e.info.signalStrength>40?r().createElement(m.A,null):e.info.signalStrength>20?r().createElement(f.A,null):r().createElement(g.A,null)}(t))));var s})))}),[j]),"stopped"==M?r().createElement("h1",null,"Enable plugin to see configuration"):r().createElement("div",null,L?r().createElement("h2",{style:"color: red;"},L.message):"",r().createElement(a.Ay,{schema:t,validator:s.Ay,onChange:e=>b(e.formData),onSubmit:({formData:e},t)=>{T(null),B("sendBaseData",e).then((e=>{200!=e.status?U(new Error(`Unable to update base data: ${e.statusText} (${e.status})`)):(I().then((e=>{N(e)})).catch((e=>{U(e)})),P())}))},onError:w("errors"),formData:v}),r().createElement("p",null),r().createElement("p",null),O.deviceCount<O.totalDevices?r().createElement(E.A,{max:O.maxTimeout,now:O.progress}):"",r().createElement("h2",null,j.size>0?" Bluetooth Devices click to configure":""),r().createElement("p",null),r().createElement("div",{style:{paddingBottom:20},class:"d-flex flex-wrap justify-content-start align-items-start"},r().createElement(p.A,{style:{maxHeight:"300px",overflowY:"auto"}},x),r().createElement("div",{style:{paddingLeft:10,paddingTop:10,display:0==Object.keys(y).length?"none":""}},r().createElement(a.Ay,{schema:y,validator:s.Ay,uiSchema:$,onChange:e=>T(e.formData),onSubmit:({formData:e},t)=>{var n;console.log(e),B("sendSensorData",n=e).then((e=>{if(200!=e.status)throw new Error(e.statusText);j.get(n.mac_address).config=n})),A({}),alert("Changes saved")},onError:w("errors"),formData:C},r().createElement("div",null,r().createElement(l.A,{direction:"row",style:{spacing:5}},r().createElement(c.A,{type:"submit",color:"primary",variant:"contained"},"Save"),r().createElement(c.A,{variant:"contained",onClick:()=>{var e;e=C.mac_address,T(j.get(e).config)}},"Undo"),r().createElement(c.A,{variant:"contained",color:"secondary",onClick:e=>{return t=C.mac_address,void(window.confirm(`Delete configuration for ${t}?`)&&function(e){try{B("removeSensorData",{mac_address:e}).then((e=>{if(200!=e.status)throw new Error(e.statusText)})),S.delete(e),_(new Map(S)),A({})}catch{}}(t));var t}},"Delete")))))))}}}]);
+"use strict";(self.webpackChunkbt_sensors_plugin_sk=self.webpackChunkbt_sensors_plugin_sk||[]).push([[893],{2995:(e,t,n)=>{n.r(t),n.d(t,{default:()=>v});var s=n(3490),a=n(4810),o=n(4147),r=n.n(o),l=n(7606),c=n(4952),i=n(3768),u=n(1431),g=n(9676),d=n(7041),m=n(3657),f=n(5027),p=n(7265),h=n(6890),E=n(8207);const w=e=>console.log.bind(console,e);var S;const v=e=>{const[t,n]=(0,o.useState)({}),[v,b]=(0,o.useState)({}),[y,D]=(0,o.useState)({}),[A,$]=(0,o.useState)({"ui:submitButtonOptions":{props:{disabled:!1,className:"btn btn-info"},norender:!0,submitText:"Submit"}}),[_,k]=(0,o.useState)([]),[x,C]=(0,o.useState)(),[T,M]=(0,o.useState)(new Map),[j,O]=(0,o.useState)({progress:0,maxTimeout:100,deviceCount:0,totalDevices:0}),[N,B]=(0,o.useState)("unknown"),[J,L]=(0,o.useState)();function U(e,t){console.log(`sending ${e}`),console.log(t);const n=new Headers;return n.append("Content-Type","application/json"),fetch(`/plugins/bt-sensors-plugin-sk/${e}`,{credentials:"include",method:"POST",body:JSON.stringify(t),headers:n})}async function P(e){return console.log(`fetching ${e}`),fetch(`/plugins/bt-sensors-plugin-sk/${e}`,{credentials:"include"})}async function z(){console.log("getProgress");const e=await P("progress");if(200!=e.status)throw new Error(`Unable get progres: ${e.statusText} (${e.status}) `);const t=await e.json();return console.log(t),t}function H(){console.log("refreshing sensor map"),async function(){console.log("getSensorData");const e=await P("sensors");if(200!=e.status)throw new Error(`Unable get sensor data: ${e.statusText} (${e.status}) `);const t=await e.json();return console.log(t),t}().then((e=>{M(new Map(e.map((e=>[e.info.mac,e]))))})).catch((e=>{L(e)}))}return(0,o.useEffect)((()=>{console.log("useEffect([])"),P("sendPluginState").then((async e=>{const t=await e.json();B(t.state),console.log("Setting up eventsource");const n=new EventSource("/plugins/bt-sensors-plugin-sk/sse");return n.addEventListener("newsensor",(e=>{console.log("newsensor");let t=JSON.parse(e.data);S.has(t.info.mac)||(console.log(`New sensor: ${t.info.mac}`),M(new Map(S.set(t.info.mac,t))))})),n.addEventListener("sensorchanged",(e=>{let t=JSON.parse(e.data);if(console.log("sensorchanged"),console.log(t),S.has(t.mac)){let e=S.get(t.mac);Object.assign(e.info,t),M(new Map(S))}})),n.addEventListener("progress",(e=>{console.log("progress");const t=JSON.parse(e.data);O(t),console.log(t)})),n.addEventListener("pluginstate",(e=>{console.log("pluginstate");const t=JSON.parse(e.data);B(t.state)})),()=>n.close()})).catch((e=>{L(e)}))}),[]),(0,o.useEffect)((()=>{console.log("useEffect([pluginState])"),"started"==N?(H(),async function(){console.log("getBaseData");const e=await P("base");if(200!=e.status)throw new Error(`Unable get base data: ${e.statusText} (${e.status}) `);const t=await e.json();return console.log(t),t}().then((e=>{n(e.schema),b(e.data)})).catch((e=>{L(e)})),z().then((e=>{O(e)})).catch((e=>{L(e)}))):(M(new Map),n({}),b({}))}),[N]),(0,o.useEffect)((()=>{console.log("useEffect([error])"),console.log(J)}),[J]),(0,o.useEffect)((()=>{console.log("useEffect([sensorMap])"),S=T,k(Array.from(T.entries()).map((e=>{const t=T.get(e[0]),n=t.config,s=Object.keys(n).length>0;return r().createElement(h.A,{action:!0,onClick:()=>{t&&(n.mac_address=e[0],D(t.schema),C(n))}},r().createElement("div",{class:"d-flex justify-content-between align-items-center",style:s?{fontWeight:"normal"}:{fontStyle:"italic"}},`${t._changesMade?"*":""}${t.info.name} MAC: ${t.info.mac} RSSI: ${a=t.info.RSSI,null==a?NaN:a}`,r().createElement("div",{class:"d-flex justify-content-between "},function(e){return null==e.info.lastContactDelta||e.info.lastContactDelta>e.config.discoveryTimeout?r().createElement(i.A,null):e.info.signalStrength>80?r().createElement(u.A,null):e.info.signalStrength>60?r().createElement(g.A,null):e.info.signalStrength>40?r().createElement(d.A,null):e.info.signalStrength>20?r().createElement(m.A,null):r().createElement(f.A,null)}(t))));var a})))}),[T]),"stopped"==N?r().createElement("h1",null,"Enable plugin to see configuration"):r().createElement("div",null,J?r().createElement("h2",{style:"color: red;"},J.message):"",r().createElement(s.Ay,{schema:t,validator:a.Ay,onChange:e=>b(e.formData),onSubmit:({formData:e},t)=>{var n;C(null),n=e,console.log("updateBaseData"),U("sendBaseData",n).then((e=>{200!=e.status?L(new Error(`Unable to update base data: ${e.statusText} (${e.status})`)):(z().then((e=>{O(e)})).catch((e=>{L(e)})),H())}))},onError:w("errors"),formData:v}),r().createElement("p",null),r().createElement("p",null),j.deviceCount<j.totalDevices?r().createElement(E.A,{max:j.maxTimeout,now:j.progress}):"",r().createElement("h2",null,T.size>0?"Bluetooth Devices click to configure":""),r().createElement("h2",null,T.size>0?"(* means sensor has unsaved changes)":""),r().createElement("p",null),r().createElement("div",{style:{paddingBottom:20},class:"d-flex flex-wrap justify-content-start align-items-start"},r().createElement(p.A,{style:{maxHeight:"300px",overflowY:"auto"}},_),r().createElement("div",{style:{paddingLeft:10,paddingTop:10,display:0==Object.keys(y).length?"none":""}},r().createElement(s.Ay,{schema:y,validator:a.Ay,uiSchema:A,onChange:e=>{T.get(e.formData.mac_address)._changesMade=!0,C(e.formData)},onSubmit:({formData:e},t)=>{var n;console.log(e),n=e,console.log("updateSensorData"),U("sendSensorData",n).then((e=>{if(200!=e.status)throw new Error(e.statusText);T.get(n.mac_address)._changesMade=!1,T.get(n.mac_address).config=n})),D({}),alert("Changes saved")},onError:w("errors"),formData:x},r().createElement("div",null,r().createElement(l.A,{direction:"row",style:{spacing:5}},r().createElement(c.A,{type:"submit",color:"primary",variant:"contained"},"Save"),r().createElement(c.A,{variant:"contained",onClick:()=>{var e;e=x.mac_address,console.log("undoChanges"),T.get(e)._changesMade=!1,C(T.get(e).config)}},"Undo"),r().createElement(c.A,{variant:"contained",color:"secondary",onClick:e=>{return t=x.mac_address,void(window.confirm(`Delete configuration for ${t}?`)&&function(e){console.log("removeSensorData");try{U("removeSensorData",{mac_address:e}).then((e=>{if(200!=e.status)throw new Error(e.statusText)})),S.delete(e),M(new Map(S)),D({})}catch{}}(t));var t}},"Delete")))))))}}}]);

package/sensor_classes/Aranet4.js

@@ -31,7 +31,7 @@ class Aranet4 extends AranetSensor{
         this.addDefaultPath("pressure","environment.pressure") 
         .read=(buff)=>{return ((buff.readUInt16LE(12)))/10}
       
-        this.addDefaultPath('relativeHumidity','environment.relative')
+        this.addDefaultPath('relativeHumidity','environment.relativeHumidity')
         .read=(buff)=>{return ((buff.readUInt8(14))/100)}
         
         this.addMetadatum('color', '',  'Warning color (G Y R)',

package/sensor_classes/GoveeH50xx.js

@@ -19,6 +19,8 @@ class GoveeH50xx extends  BTSensor {
 
     initSchema(){
         super.initSchema()
+        this.addDefaultParam("zone")
+
         this.addDefaultPath("temp","environment.temperature")
             .read= (buffer)=>{return 273.15+(buffer.readUInt16LE(1)/100) }
         

package/sensor_classes/MopekaTankSensor.js

@@ -335,8 +335,8 @@ class MopekaTankSensor extends BTSensor{
         if (this.name)
             return this.name
 
-        const _name = MopekaDevice.get(this?.modelID??0x0).name
-        return _name?_name:MopekaDevice.get(0x0).name
+        const _name = MopekaDevices.get(this?.modelID??0x0).name
+        return _name?_name:MopekaDevices.get(0x0).name
         
     }
 }

package/sensor_classes/UltrasonicWindMeter.js

@@ -12,6 +12,9 @@ class UltrasonicWindMeter extends BTSensor{
     hasGATT(){
         return true
     }
+    usingGATT(){
+        return true
+    }
     emitGATT(){
         this.battCharacteristic.readValue()
         .then((buffer)=>
@@ -29,6 +32,7 @@ class UltrasonicWindMeter extends BTSensor{
     }
      initSchema(){
         super.initSchema()
+        this.getGATTParams()["useGATT"].default=true
         this.addDefaultPath("batt",'sensors.batteryStrength')
             .read=(buffer)=>{return (buffer.readUInt8())/100}
 

package/sensor_classes/VictronSmartBatteryProtect.js

@@ -26,6 +26,7 @@ class VictronSmartBatteryProtect extends VictronSensor{
 
     initSchema(){
         super.initSchema()
+        this.addDefaultParam("id")
         this.addMetadatum('deviceState','', 'device state', 
             (buff)=>{return VC.OperationMode.get(buff.readUInt8(1))})
         this.addMetadatum('outputStatus','', 'output status', //TODO

package/src/components/PluginConfigurationPanel.js

@@ -5,8 +5,7 @@ import {useEffect, useState} from 'react'
 
 import {Button, Grid} from '@material-ui/core';
 
-
-import {  SignalCellular0Bar, SignalCellular1Bar, SignalCellular2Bar, SignalCellular3Bar, SignalCellular4Bar, SignalCellularConnectedNoInternet0Bar    } from '@material-ui/icons';
+import { SignalCellular0Bar, SignalCellular1Bar, SignalCellular2Bar, SignalCellular3Bar, SignalCellular4Bar, SignalCellularConnectedNoInternet0Bar    } from '@material-ui/icons';
 
 const log = (type) => console.log.bind(console, type);
 
@@ -74,7 +73,7 @@ export default (props) => {
   }
 
   async function getSensorData(){
-  
+    console.log("getSensorData")
     const response = await fetchJSONData("sensors")
     if (response.status!=200){
       throw new Error(`Unable get sensor data: ${response.statusText} (${response.status}) `)
@@ -86,7 +85,7 @@ export default (props) => {
   }
 
   async function getBaseData(){
-    
+    console.log("getBaseData")
     const response = await fetchJSONData("base")
     if (response.status!=200){
       throw new Error(`Unable get base data: ${response.statusText} (${response.status}) `)
@@ -96,7 +95,7 @@ export default (props) => {
     return json
   }
   async function getProgress(){
-    
+    console.log("getProgress")
     const response = await fetchJSONData("progress")
     if (response.status!=200){
       throw new Error(`Unable get progres: ${response.statusText} (${response.status}) `)
@@ -107,21 +106,26 @@ export default (props) => {
   }
 
   function updateSensorData(data){
+    console.log("updateSensorData")
     sendJSONData("sendSensorData", data).then((response)=>{ 
       if (response.status != 200) {
         throw new Error(response.statusText)
       }
-      debugger
+      sensorMap.get(data.mac_address)._changesMade=false
       sensorMap.get(data.mac_address).config = data
       
     })
-  }
+  } 
 
   function undoChanges(mac) {
+    console.log("undoChanges")
+    sensorMap.get(mac)._changesMade = false
     setSensorData( sensorMap.get(mac).config )
   }
 
   function removeSensorData(mac){
+    console.log("removeSensorData")
+
     try{ 
     
       sendJSONData("removeSensorData", {mac_address:mac} ).then((response)=>{
@@ -142,6 +146,8 @@ export default (props) => {
 
 
   function updateBaseData(data){
+    console.log("updateBaseData")
+
     sendJSONData("sendBaseData", data).then( (response )=>{
       if (response.status != 200) {
         setError(new Error(`Unable to update base data: ${response.statusText} (${response.status})`))
@@ -170,7 +176,7 @@ export default (props) => {
 
 
   useEffect(()=>{
-
+    console.log("useEffect([])")
     fetchJSONData("sendPluginState").then( async (response)=> {
       const json = await response.json()
       setPluginState(json.state)
@@ -178,6 +184,7 @@ export default (props) => {
       const eventSource = new EventSource("/plugins/bt-sensors-plugin-sk/sse")
       
       eventSource.addEventListener("newsensor", (event) => {
+        console.log("newsensor")
         let json = JSON.parse(event.data)
         
         if (!_sensorMap.has(json.info.mac)) {
@@ -187,7 +194,9 @@ export default (props) => {
       });
 
       eventSource.addEventListener("sensorchanged", (event) => {
-        let json = JSON.parse(event.data)        
+        let json = JSON.parse(event.data)      
+        console.log("sensorchanged")
+        console.log(json)
         
         if (_sensorMap.has(json.mac)) {      
           let sensor = _sensorMap.get(json.mac)
@@ -196,12 +205,14 @@ export default (props) => {
         }
       });
       eventSource.addEventListener("progress", (event) => {
+        console.log("progress")
         const json = JSON.parse(event.data)  
         setProgress(json)
         console.log(json)
       });
 
       eventSource.addEventListener("pluginstate", (event) => {
+        console.log("pluginstate")
         const json = JSON.parse(event.data)  
         setPluginState(json.state)
       });
@@ -215,6 +226,7 @@ export default (props) => {
 },[])
 
 useEffect(()=>{
+  console.log("useEffect([pluginState])")
   if (pluginState=="started"){
     refreshSensors()
     
@@ -240,6 +252,7 @@ useEffect(()=>{
 },[pluginState])
 
 useEffect(()=>{
+  console.log("useEffect([error])")
   console.log(error)
 },[error])
 
@@ -269,6 +282,8 @@ function signalStrengthIcon(sensor){
 
 }
 useEffect(()=>{
+  console.log("useEffect([sensorMap])")
+
     _sensorMap = sensorMap
     
     setSensorList(
@@ -289,7 +304,7 @@ useEffect(()=>{
         }
         }> 
         <div  class="d-flex justify-content-between align-items-center" style={hasConfig?{fontWeight:"normal"}:{fontStyle:"italic"}}>
-        {`${sensor.info.name} MAC: ${sensor.info.mac} RSSI: ${ifNullNaN(sensor.info.RSSI)}`  }
+        {`${sensor._changesMade?"*":""}${sensor.info.name} MAC: ${sensor.info.mac} RSSI: ${ifNullNaN(sensor.info.RSSI)}`  }
         <div class="d-flex justify-content-between ">
           {
             signalStrengthIcon(sensor)
@@ -329,7 +344,8 @@ useEffect(()=>{
                      now={progress.progress} 
         />:""
       }
-      <h2>{`${sensorMap.size>0?" Bluetooth Devices click to configure":""}`}</h2>
+      <h2>{`${sensorMap.size>0?"Bluetooth Devices click to configure" :""}`}</h2>
+      <h2>{`${sensorMap.size>0?"(* means sensor has unsaved changes)" :""}`}</h2>
       <p></p>
       <div style={{paddingBottom: 20}} class="d-flex flex-wrap justify-content-start align-items-start">
       <ListGroup style={{  maxHeight: '300px', overflowY: 'auto' }}>
@@ -340,7 +356,14 @@ useEffect(()=>{
         schema={schema}
         validator={validator}
         uiSchema={uiSchema}
-        onChange={(e) => setSensorData(e.formData)}
+        onChange={(e) => {
+            const s = sensorMap.get(e.formData.mac_address)
+            s._changesMade=true
+            //s.config = e.formData; 
+
+            setSensorData(e.formData)
+          }
+        }
         onSubmit={({ formData }, e) => {
           console.log(formData) 
           updateSensorData(formData)