RobertD502 / home-assistant-petkit

Home Assistant integration for PetKit devices
MIT License
213 stars 32 forks source link

feat: add cloud device status for LitterBox #97

Open Jezza34000 opened 1 week ago

Jezza34000 commented 1 week ago
RobertD502 commented 1 week ago

Hi, please revert the changes that you have made to the strings and English translation file as it will break the integration for feeders. Instead, create the device_status (without the on batteries state) as its own key while keeping the feeder_status as it was.

Edit: disregard, I missed the commit you made that fixed the issue.

Jezza34000 commented 1 week ago

So yes, I chose to reuse "feeder_status" which I renamed to "device_status" to avoid making the code heavier. There is actually no "battery" status on the Litter Boxes but that is not that problematic. What do you think?