search

Taraman17 / hass-homee

a Home Assistant custom component to integrate the homee smart home platform
MIT License
14 stars 2 forks source link

Unexpected behaviour when more than one homee is configured #5

Open Taraman17 opened 9 months ago

Taraman17 commented 9 months ago

Type of problem

Version

all

Detailed description

Currently it is not possible to use more than one configured homee at the same time. Only devices from one homee will work as expected. Removing one of the configurations will cause the other to break until HA is restarted.

Notes

The problem occurs because the homee.set_value service is registered/unregistered for every config entry and does not take into account the homee cube that should be targeted.

ikrentchev commented 9 months ago

Have you tried linking the homee cubes to one another and using just one in homeassistant integration? One acting as master and the other as an addition. This is my plan and I wanted to know if someone tested it this way.

Taraman17 commented 9 months ago

No, I only have one homee and just copied this issue from the original repo, since I'm taking over code-ownership for this integration. So I can't test that case.

Taraman17 commented 9 months ago

Hint on how to fix this: https://github.com/wlcrs/huawei_solar/commit/1b39063e9e37395d278332bd63ad6cf77d084d21

Taraman17 commented 4 months ago

If there is any user here who has 2 Homees, I would love to know, if this issue is still existant after the changes made in the last months.

ikrentchev commented 4 months ago

When I saw the limitation for just one homee, I was thinking to bind the second one, when I buy it, to the first one and present the additional garage door via the first and only connected to HA homee. Now, I saw that you are advancing with the integration to allow a second homee device to be connected and I promise to try it, when the time comes but maybe my first idea is also an option. What do you think?

Best, Ivan

On Tue, Feb 13, 2024 at 17:31 Markus Adrario @.***> wrote:

If there is any user here who has 2 Homees, I would love to know, if this issue is still existant after the changes made in the last months.

— Reply to this email directly, view it on GitHub https://github.com/Taraman17/hass-homee/issues/5#issuecomment-1941795137, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6WTA7FWKFCDEH2QXIKW2J3YTOBNHAVCNFSM6AAAAAA4TXQQXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBRG44TKMJTG4 . You are receiving this because you commented.Message ID: @.***>

Taraman17 commented 4 months ago

It is definitely an option. But I think the integration should be capable of handling more than one homee anyhow.

ikrentchev commented 4 months ago

Thanks for everything that you do for all of us! Much appreciated!

On Tue, Feb 13, 2024 at 10:29 PM Markus Adrario @.***> wrote:

It is definitely an option. But I think the integration should be capable of handling more than one homee anyhow.

— Reply to this email directly, view it on GitHub https://github.com/Taraman17/hass-homee/issues/5#issuecomment-1942417220, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6WTA7CI5CGJPKVCC44LXMDYTPEMDAVCNFSM6AAAAAA4TXQQXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBSGQYTOMRSGA . You are receiving this because you commented.Message ID: @.***>