bmccluskey / robovac

Add a Eufy RoboVac easily to Home Assistant
Other
116 stars 27 forks source link

The robovac custom integration is using deprecated vacuum feature #39

Open darekashiranai opened 1 year ago

darekashiranai commented 1 year ago

The custom integration robovac is extending the deprecated base class VacuumEntity instead of StateVacuumEntity.

Please report it to the author of the robovac custom integration.

Once an updated version of robovac is available, install it and restart Home Assistant to fix this issue.

pem884 commented 1 year ago

Thanks for creating an issue; this is a duplicate of #31, and #38 also addresses it. The solution is to try the forks referenced in those issues. Here's the text from @CodeFoodPixels in #38:

Given that the maintainer of this integration hasn't been active on GitHub since December 2022, I've forked this repo with a view to maintain it along with anyone else who would like to help.

I'm also submitting my fork to be included in HACS by default.

Fork is here, but happy to consider setting up an org or whatever: https://github.com/CodeFoodPixels/robovac

I also noticed that this is your first issue, so you may be new to picking up on some clues about this being a duplicate issue. I recommend searching existing issues within the repo if you haven't already before adding a new issue. Also, if you're new to using GitHub, I suggest that seeing an open issue like #38 is a red flag and could guide you to the answer on your own.

Hope you find this helpful!

DanDixon commented 1 year ago

This fixes it, but hasn't been merged in yet: https://github.com/rustymyers/robovac/tree/main

You can just add this repo to HACS, install, and restart, and you should be good to go.

I had to make a couple of small updates to my scripts as the status attribute (cleaning, idle, etc) was removed and is now just the base state of the vacuum.

michguy2000 commented 1 year ago

Thanks for creating an issue; this is a duplicate of #31, and #38 also addresses it. The solution is to try the forks referenced in those issues. Here's the text from @CodeFoodPixels in #38:

Given that the maintainer of this integration hasn't been active on GitHub since December 2022, I've forked this repo with a view to maintain it along with anyone else who would like to help. I'm also submitting my fork to be included in HACS by default. Fork is here, but happy to consider setting up an org or whatever: https://github.com/CodeFoodPixels/robovac

I also noticed that this is your first issue, so you may be new to picking up on some clues about this being a duplicate issue. I recommend searching existing issues within the repo if you haven't already before adding a new issue. Also, if you're new to using GitHub, I suggest that seeing an open issue like #38 is a red flag and could guide you to the answer on your own.

Hope you find this helpful!

Thank you for the HACS fix. Much appreciated (through paypal) . Regards