Update innr.ts

[z647230]

added support for innr sp-240 device:

    zigbeeModel: ['SP 240'],
    model: 'SP 240',
    vendor: 'Innr',

[z647230]

Hm, configure did not cause issues in my initial code but it does with the new extend setup. I will have a look and capture the error messages.Grtz, MartinOn 7 Dec 2023, at 20:00, Koen Kanters ***@***.***> wrote: @Koenkk commented on this pull request. In src/devices/innr.ts:

@@ -684,6 +685,30 @@ const definitions: Definition[] = [

} }, }, + { This should already be fixed in the dev branch Does the configure also succeed? Press the yellow refresh icon in the device about page. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>

[Koenkk]

what error message does show up?

[z647230]

Error 2023-12-08 20:02:22Failed to configure '0xa4c13899b37f4740', attempt 4 (Error: Read 0xa4c13899b37f4740/1 seMetering(["divisor","multiplier"], {"sendWhen":"immediate","timeout":10000,"disableResponse":false,"disableRecovery":false,"disableDefaultResponse":true,"direction":0,"srcEndpoint":null,"reservedBits":0,"manufacturerCode":null,"transactionSequenceNumber":null,"writeUndiv":false}) failed (Status 'UNSUPPORTED_ATTRIBUTE')
at Endpoint.checkStatus (/app/node_modules/zigbee-herdsman/src/controller/model/endpoint.ts:421:28) at Endpoint.read (/app/node_modules/zigbee-herdsman/src/controller/model/endpoint.ts:563:22) at configure (/app/node_modules/zigbee-herdsman-converters/src/lib/modernExtend.ts:135:25) at Object.configure (/app/node_modules/zigbee-herdsman-converters/index.js:128:25) at Configure.configure (/app/lib/extension/configure.ts:117:13) at Configure.onMQTTMessage (/app/lib/extension/configure.ts:55:21))

[Koenkk]

Thanks!

[z647230]

You’re welcome.But like said before, the powerInBehavior does work correctly so no need to disable it.Rgdz, MartinGrtz, MartinOn 10 Dec 2023, at 20:50, Koen Kanters ***@***.***> wrote: Thanks! —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>

[Koenkk]

It's disabled:

zigbee-herdsman-converters/src/devices/innr.ts

Line 668 in c7ee833

onOff({powerOnBehavior: false}),

[z647230]

Yes, I noticed but it should be enabled as the powerOnBehavior is supported correctly by the device.I fully tested it with all the states.Grtz, MartinOn 11 Dec 2023, at 19:22, Koen Kanters ***@***.***> wrote: It's disabled: https://github.com/Koenkk/zigbee-herdsman-converters/blob/c7ee83322f9cd4f5c83da7f042e2a5ae5468ec09/src/devices/innr.ts#L668 —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>

[Koenkk]

I see, enabled it!

Changes will be available in the dev branch in a few hours from now.