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

Integration stopped working #218

Open
mash727 opened this issue Jul 9, 2024 · 10 comments
Open

Integration stopped working #218

mash727 opened this issue Jul 9, 2024 · 10 comments

Comments

@mash727
Copy link

mash727 commented Jul 9, 2024

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
Copy link

Same problem over here. Have you found a solution?

@zwizwi
Copy link

zwizwi commented Aug 19, 2024

Same problem here!

@zwizwi
Copy link

zwizwi commented Aug 25, 2024

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

@grm
Copy link
Contributor

grm commented Sep 2, 2024

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
Copy link

zwizwi commented Sep 3, 2024

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

It‘s the version 1.1.1 rc5

image

@seb-5x
Copy link

seb-5x commented Sep 3, 2024 via email

@JayChickenK
Copy link

JayChickenK commented Sep 3, 2024

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
Copy link
Contributor

grm commented Sep 3, 2024

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

@JayChickenK
Copy link

JayChickenK commented Sep 3, 2024

@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 #216 (comment)

@zwizwi
Copy link

zwizwi commented Sep 4, 2024

You are right, the rc3 is working perfectly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants