-
Notifications
You must be signed in to change notification settings - Fork 53
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
Integration stopped working #218
Comments
Same problem over here. Have you found a solution? |
Same problem here! |
Have done the updates of homeassistant and hacs and the integration works as before! |
Helloo, sorry for the delay in the answer. |
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
<#218 (comment)>,
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>
|
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. |
@JayChickenK maybe you could make a PR with the code you adapted in order to merge it to master ? |
@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: Originally posted by @crazyfx1 in #216 (comment) |
You are right, the rc3 is working perfectly! |
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
The text was updated successfully, but these errors were encountered: