Merge pull request #7 from CMLabsLLC/verified2

Additional Verification Changes
CMLabsLLC · Oct 8, 2024 · fd9c3f6 · fd9c3f6
2 parents 0d13ae4 + 7add372
commit fd9c3f6
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 6 deletions.
diff --git a/config.schema.json b/config.schema.json
@@ -8,7 +8,7 @@
       "name": {
         "title": "name",
         "type": "string",
-        "required": false,
+        "required": true,
         "default": "Car Lock"
       },
       "username": {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -2,7 +2,7 @@
   "name": "homebridge-subaru",
   "displayName": "Homebridge Subaru",
   "type": "module",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Exposes basic starlink funcationality to homekit",
   "author": "Russell Stephens",
   "license": "Apache-2.0",
@@ -17,7 +17,10 @@
   "keywords": [
     "homebridge-plugin",
     "subaru",
-    "starlink"
+    "starlink",
+    "homebridge plugin",
+    "homebridge subaru",
+    "homebridge subaru starlink"
   ],
   "main": "dist/index.js",
   "engines": {

diff --git a/src/subaruHomebridgePlatform.ts b/src/subaruHomebridgePlatform.ts
@@ -27,6 +27,10 @@ export class SubaruHomebridgePlatform implements DynamicPlatformPlugin {
     this.Service = api.hap.Service;
     this.Characteristic = api.hap.Characteristic;
 
+    if (!config.name) {
+      this.log.error('Missing required config value: name');
+    }
+
     if (!config.username) {
       this.log.error('Missing required config value: username');
     }
@@ -83,7 +87,7 @@ export class SubaruHomebridgePlatform implements DynamicPlatformPlugin {
 
     const existingAccessory = this.accessories.find(accessory => accessory.UUID === uuid);
 
-    const deviceName = this.config.name || 'Subaru Door Lock';
+    const deviceName = this.config.name || '';
 
     const device = {
       name: deviceName,