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

Water Control Bluetooth #28545

Closed
Des2323 opened this issue Aug 11, 2023 · 4 comments · Fixed by home-assistant/core#98282
Closed

Water Control Bluetooth #28545

Des2323 opened this issue Aug 11, 2023 · 4 comments · Fixed by home-assistant/core#98282

Comments

@Des2323
Copy link

Des2323 commented Aug 11, 2023

Feedback

Thanks for this addon, I just bought one.
Mine is type 01889-29, but it also works perfectly.
And yes, if I use the official Android App, the connection is lost, it only works after Factory Reset, turning off Bluetooth on the phone didn't help me.

Is there a way to make the connected soil moisture sensor (1867-20) also work?
How can I help you with this?

Des

URL

https://www.home-assistant.io/integrations/gardena_bluetooth/

Version

2023.8.2

Additional information

No response

@home-assistant
Copy link

Hey there @elupus, mind taking a look at this feedback as it has been labeled with an integration (gardena_bluetooth) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of gardena_bluetooth can trigger bot actions by commenting:

  • @home-assistant close Closes the feedback.
  • @home-assistant rename Awesome new title Renames the feedback.
  • @home-assistant reopen Reopen the feedback.
  • @home-assistant unassign gardena_bluetooth Removes the current integration label and assignees on the feedback, add the integration domain after the command.

@elupus
Copy link
Contributor

elupus commented Aug 11, 2023

The soil sensor should be easy to add i think: we need to add entities for some of these characteristics: https://github.com/elupus/gardena-bluetooth/blob/c8475b8f72a4c0f4b84c96316a670b62952e6111/gardena_bluetooth/const.py#L76

They may need to condition their availability on connected_state attribute.

They should likely be added here: https://github.com/home-assistant/core/blob/4342a95be0bde6bcc7ac9471c9b01f4b41b8f96d/homeassistant/components/gardena_bluetooth/sensor.py#L37

Ps. Since i don't have one, i'm hesitant to add this myself. Would be better if somebody with the sensor tried it first.

@Des2323
Copy link
Author

Des2323 commented Aug 11, 2023

That sounds good, I have one :)
I think the first 8 characters should be used for CharacteristicInt, something like 98bd0xxx if I'm correct.
How can I get this?

I'm not a programmer (I'm an administrator), I can only "read" program code, but I absolutely cannot program in Python.
But with minor or major help, I can run all kinds of programs if necessary :)
And then I'll be happy to try it

@elupus
Copy link
Contributor

elupus commented Aug 11, 2023

This is mostly a feature request, so i'll close it for now. I've opened up a pull request with sensor support here: home-assistant/core#98282 if you can test that we might be able to merge it.

You can grab this directory: https://github.com/elupus/home-assistant/tree/gardena/moisture/homeassistant/components/gardena_bluetooth and add as a custom component or you can run the branch fully if you know how to do that.

@elupus elupus closed this as completed Aug 15, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Sep 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants