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

Use connections from device config to setup hass devices #29

Conversation

casparz
Copy link

@casparz casparz commented Jan 8, 2024

My take on the setup of entities and devices based on the physical configuration of the B+ device. I welcome any remarks, since it is my first contribution to an integration (and my python is a bit rusty).

closes #12

@koenhendriks
Copy link
Owner

Thanks for the PR! I'll try to find some time in the upcoming week to review this!

@koenhendriks koenhendriks self-assigned this Jan 14, 2024
@koenhendriks koenhendriks self-requested a review January 14, 2024 12:53
@koenhendriks koenhendriks added the enhancement New feature or request label Jan 14, 2024
Copy link
Owner

@koenhendriks koenhendriks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did some small formatting changes but kept your logic.

I've tested this on my button+ and it seems to work perfectly, thank you very much :)

@koenhendriks koenhendriks merged commit 8560726 into koenhendriks:main Jan 14, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use connections from device config to setup hass devices
2 participants