dave-code-ruiz / elkbledom

Home Assistant custom component for LED STRIP NAME ELK BLEDOM
MIT License
77 stars 16 forks source link

Light strip is not detected as a device #34

Closed dwaned closed 1 year ago

dwaned commented 1 year ago

Home Assistant 2023.8.0 Supervisor 2023.07.1 Operating System 10.4 Frontend 20230802.0 - latest

Added integration via HACS with latest version, and when adding device, it finds the lights, blink test is successful, but the lights show as entity, unable to control.

image
dave-code-ruiz commented 1 year ago

Ok probably you have and error in your log

Please post error here.

If you dont know how to, please search how to in other issue i explain it

doudigit commented 1 year ago

@dave-code-ruiz i got the same problem. Here is the log

` Logger: homeassistant.components.light Source: helpers/entity_platform.py:619 Integration: Light (documentation, issues) First occurred: 05:36:20 (6 occurrences) Last logged: 05:55:55

Error adding entities for domain light with platform elkbledom Error while setting up elkbledom platform for light Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 510, in async_add_entities await asyncio.gather(*tasks) File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 619, in _async_add_entity device = dev_reg.async_get(self.hass).async_get_or_create( ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TypeError: homeassistant.helpers.device_registry.DeviceRegistry.async_get_or_create() got multiple values for keyword argument 'config_entry_id' `

tracedebrake commented 1 year ago

Same as #32

LexMrack commented 1 year ago

After updating HA to 2023.8.0, I had the same thing. After the rollback to 2023.7, everything came back. После обновления HA на 2023.8.0 у меня было тоже самое. После отката на 2023,7 вернулось всё обратно.

tracedebrake commented 1 year ago

image

There was a change in 2023.8; he must adapt his code to https://github.com/home-assistant/core/pull/95547

It'll be broken as long as he or someone else fix it. Might give it a shot.

dave-code-ruiz commented 1 year ago

Sorry i am on vacation in August

Bad month for broke core code

May be fix it in long time, Sorry for the inconvenience.

dave-code-ruiz commented 1 year ago

I can accept an other user PR if someone do it

tracedebrake commented 1 year ago

Change proposed in light.py

That function is meant to return a DeviceInfo as defined here https://github.com/home-assistant/core/blob/614f3c6a15713aa762269f7c628d4420987cbd4c/homeassistant/helpers/entity.py#L177 So it was probably working by accident before

tracedebrake commented 1 year ago

OK everybody, issue is fixed. Debugged it on a sunday morning on the toilet so you all owe me a beer.

@dave-code-ruiz can close #32 and #34

Might take a look at #33 soon.

Have a great summer everybody.

dwaned commented 1 year ago

Thank you! Fix is confirmed 🍻 🍺

dave-code-ruiz commented 1 year ago

@tracedebrake thnks for your help in my vacations

dave-code-ruiz commented 1 year ago

Thnks @dwaned too for your support