Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Two B+'s, cannot assign B+ to different areas #36

Closed
wimjanse opened this issue Jan 15, 2024 · 7 comments · Fixed by #40 or #46
Closed

Two B+'s, cannot assign B+ to different areas #36

wimjanse opened this issue Jan 15, 2024 · 7 comments · Fixed by #40 or #46
Assignees
Labels
bug Something isn't working

Comments

@wimjanse
Copy link
Contributor

Describe the bug
I added first B+ with ID 45_a450 to HA, area: "Studeerkamer". Then added second B+ with ID 45_a5d0, during addition I see the Display and 3 BARS, already with area "Studeerkamer". When I make a change in area ("Huiskamer") during the addition of the second B+, also the first B+ changes to "Huiskamer"

To Reproduce
Steps to reproduce the behavior:

  • Add device B+ nr.1 to Integration, give IP address of B+ no. 1 and thereafter enter area for Display, Bar 1/2/3.
  • Add device B+ nr.2 to Integration, give IP address of B+ no. 2, areas for Display and Bar 1/2/3 now show area from no.1
  • During this addition, change are for no.2
  • Also area for no.1 is changed to same as given above for no.2

Expected behavior
Different areas for the two B+'s

@wimjanse wimjanse added the bug Something isn't working label Jan 15, 2024
@nvdweem
Copy link

nvdweem commented Jan 18, 2024

I might be having the same issue, but my devices aren't entirely grouped together.

I can set labels for the (3 in my case) devices but I had to rename them all to be sure which one is which as they all show up in the same place. The Integration page does correctly show the separate devices, but choosing 1 device shows all same-bars combined in 1 device.
image

Here you see the 3 devices grouped together. The devices have their names changed, that made them group up nicely. Before that it was all a mess.
image

These haven't gotten a new name, it's completely unclear which label changes what.
image

@koenhendriks
Copy link
Owner

koenhendriks commented Jan 24, 2024

This seems like an error in how the ButtonPlusHub class is currently setup. There is only 1 instance but it is carrying logic for multiple devices. I need to look into changing this and maybe create a sub device for the base module to split each physical device on (button bars and display).

It's a bit hard to test this as I've only a single ButtonPlus to develop with.

@koenhendriks koenhendriks linked a pull request Jan 24, 2024 that will close this issue
@koenhendriks
Copy link
Owner

I've added the hub id as a prefix for the child devices' identifiers in 0.0.8 this might potential fix this issue. Please remove the integration and entities manually before upgrading and after the upgrade please try again @wimjanse @nvdweem

@nvdweem
Copy link

nvdweem commented Jan 24, 2024

I removed the entities and updated the integration, issue was not fixed.

image

@wimjanse
Copy link
Contributor Author

Issue not fixed

Same here, manually removed both entities, updated and reinstalled.
After B+ nr 1, I assigned area "Studeerkamer" to Display and Bars 1, 2 and 3.
Thereafter, added B+ nr 2, I assigned area "Huiskamer" to Display and Bars 1, 2 and 3. Area's of B+ nr 1 are now also indicated as "Huiskamer", so cannot distinguish between both B+'s.

@koenhendriks
Copy link
Owner

Hey guys, Sorry for the late response I was on holiday. I will reopen this issue.

I will soon receive a second Button+ device so then I'll be able to better debug this issue and hopefully come up with a solution soon.

@koenhendriks koenhendriks reopened this Feb 12, 2024
@wimjanse
Copy link
Contributor Author

wimjanse commented Feb 12, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants