markvader / sonic

Sonic water shutoff valve Home Assistant integration by @markvader
MIT License
14 stars 3 forks source link

Temperature and pressure #26

Closed marcgeurts closed 1 year ago

marcgeurts commented 1 year ago

I notice that you already added the code for fetching the water temperature and pressure, but this doesn't show how up in my home assistant instance, is this part already working?

markvader commented 1 year ago

It should be. Assuming your sonic is reporting the values to the Hero Lab server. Try reloading the integration to see if it fixes it. Screenshot_20230802-175431.png

marcgeurts commented 1 year ago

Thanks for your fast response, I tried doing this several times already, still only get these entities:

image

Do i need to do something additional apart from just setting it up in the Sonic app?

markvader commented 1 year ago

Do you have one or two devices listed under the Sonic integration. I set them up with two devices (one for the device itself and one for the settings).

Screenshot_20230803-124653.png

marcgeurts commented 1 year ago

No only 1 device:

image

I just did a factory reset on the device, removed the property from my account and added a new one with a different name, I can see the new name in home assistant, but still no device values.

The sonic app itself is working as expected (I can see all the events and other values)

marcgeurts commented 1 year ago

It's is the new pro model (with the black button), can it be caused by that?

image

I do get a device token with: user_info = await client.user.async_get_user_details()

The following does however not return any data: sonic_data = await client.sonic.async_get_all_sonic_details()

When i do a call to https://api.hero-labs.com/ape/v1/sonics, i also don't get any data returned: image

marcgeurts commented 1 year ago

Issue has been resolved, turns out i have the Wi-Fi model.... i will just have to be patient till hero-labs is finished with their HA integration (which they are working on)

marcgeurts commented 1 year ago

Maybe add to the documentation that only the model with wall plug is supported for now? (expected to change by the end of the year)