py-smart-gardena / hass-gardena-smart-system

Home Assistant custom component integration for Gardena Smart System
Apache License 2.0
213 stars 53 forks source link

Integration stopped working #218

Open mash727 opened 4 months ago

mash727 commented 4 months ago

I have a similar problem that the integration stopped working. Even deleting, creating new tokens does not help. Integration simply sits in an initialisation mode and eventually throws up a failure. Is there anyone that has taken a fork that would be prepared to share code. Am happy to act as a tester. It seems this super code is no longer supported

Arjan1993 commented 3 months ago

Same problem over here. Have you found a solution?

zwizwi commented 3 months ago

Same problem here!

zwizwi commented 3 months ago

Have done the updates of homeassistant and hacs and the integration works as before!

grm commented 2 months ago

Helloo,

sorry for the delay in the answer. Is it wokring now ? You should update and use beta release until I relaese 1.1.1. ;)

zwizwi commented 2 months ago

The irrigation don‘t have the humidity nor the moisture data.

It‘s the version 1.1.1 rc5

image

seb-5x commented 2 months ago

Hello I confirm same pb with rc5

Le mar. 3 sept. 2024, 07:01, zwizwi @.***> a écrit :

The irrigation don‘t have the humidity nor the moisture data.

It‘s the version 1.1.1 rc5

image.png (view on web) https://github.com/user-attachments/assets/28b3d3ce-6afd-4411-8979-9e8918e7e11e

— Reply to this email directly, view it on GitHub https://github.com/py-smart-gardena/hass-gardena-smart-system/issues/218#issuecomment-2325606004, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUTBQC3AF2FQBUBU6234TWDZUU7DHAVCNFSM6AAAAABKSC42A2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRVGYYDMMBQGQ . You are receiving this because you are subscribed to this thread.Message ID: <py-smart-gardena/hass-gardena-smart-system/issues/218/2325606004@ github.com>

JayChickenK commented 2 months ago

with #216 SensorDeviceClass was changed, i tried that change before with the same result that both humidity and moisture data missing in homeassistant. But i didnt tried rc5 yet, will try it today after work, right now im still on a modified rc3.

grm commented 2 months ago

@JayChickenK maybe you could make a PR with the code you adapted in order to merge it to master ? WDYT ?

JayChickenK commented 2 months ago

@grm crazyfx1 has already described the problem in great detail. I would suggest reverting the pull request until the library is adapted. I have only implemented the sensible open pull requests in my rc3, nothing new nor fancy. In this case is no pr needed.

_The py-smart-gardena lib does not return "soil_moisture" but still "soil_humidity". Either change it back or adapt the library and push a new version: https://github.com/py-smart-gardena/py-smart-gardena/blob/master/src/gardena/devices/soil_sensor.py_

Originally posted by @crazyfx1 in https://github.com/py-smart-gardena/hass-gardena-smart-system/issues/216#issuecomment-2326269348

zwizwi commented 2 months ago

You are right, the rc3 is working perfectly!