search

senorshaun / homebridge-radiora2

Lutron RadioRA2 plugin for homebridge
MIT License
10 stars 5 forks source link

GOT SIGTERM, shutting down home bridge issue #12

Closed jfinestone closed 2 years ago

jfinestone commented 2 years ago

Added a keypad to my previously working radiora1/homebridge integration. and then this started...

[12/30/2021, 9:31:21 AM] TypeError: Cannot read properties of undefined (reading 'toString') at new KeypadButton (/homebridge/node_modules/homebridge-radiora2/lib/accessories/keypadbutton.js:15:72) at RadioRA2Platform.addKeypadButtonAccessory (/homebridge/node_modules/homebridge-radiora2/index.js:141:44) at RadioRA2Platform. (/homebridge/node_modules/homebridge-radiora2/index.js:281:34) at Array.forEach () at RadioRA2Platform. (/homebridge/node_modules/homebridge-radiora2/index.js:272:26) at RadioRa2.emit (node:events:390:28) at incomingData (/homebridge/node_modules/homebridge-radiora2/lib/radiora2.js:97:20) at Socket. (/homebridge/node_modules/homebridge-radiora2/lib/radiora2.js:27:92) at Socket.emit (node:events:390:28) at addChunk (node:internal/streams/readable:315:12) [12/30/2021, 9:31:21 AM] [Hubitat-v2] Temperature Unit is Now: (F) [12/30/2021, 9:31:21 AM] [Hubitat-v2] Updating Global Values | HubIP: 10.100.10.28 | UsingCloud: false [12/30/2021, 9:31:21 AM] [Hubitat-v2] Devices to Remove: (0) [12/30/2021, 9:31:21 AM] [Hubitat-v2] Devices to Update: (4) [12/30/2021, 9:31:21 AM] [Hubitat-v2] Devices to Create: [12/30/2021, 9:31:41 AM] Got SIGTERM, shutting down Homebridge... [12/30/2021, 9:31:41 AM] [RadioRA2] Disconnecting

senorshaun commented 2 years ago

the button isn't named in your config. handled with the pull request which makes those optional

jfinestone commented 2 years ago

Thank you for the prompt response. Can you explain a little further I don’t see where I had to name it and I’m not exactly sure what a pull request is or how to use that.

Thanks.

On Fri, Dec 31, 2021 at 5:27 PM senorshaun @.***> wrote:

the button isn't named in your config. handled with the pull request which makes those optional

— Reply to this email directly, view it on GitHub https://github.com/senorshaun/homebridge-radiora2/issues/12#issuecomment-1003481067, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHS7I7J655SQVR4UJKCJY3UTZKBNANCNFSM5LAB37UQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

senorshaun commented 2 years ago

I just published 1.3.4 to npm. update your copy of the plugin. @cariacou modified some code to make parameters, like the name of buttons, optional. here's an example config where it shows buttons and keypads using names "keypads": [ { "name": "Lutron Buttons", "id": 1, "model": "RR-MAIN-REP-WH", "serial": "xxxxxxxxx", "stateless": false, "buttons":[ { "name": "Normal Light Level", "id": 1, "led": 101 }, { "name": "Low Light Level", "id": 2, "led": 102 }, ...