-
Notifications
You must be signed in to change notification settings - Fork 907
Adds support for Catit Pixi Smart Feeder (Model: 43752) #3889
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
Adds support for Catit Pixi Smart Feeder (Model: 43752) #3889
Conversation
colin-nolan
commented
Oct 25, 2025
- Catit branded pet feeder was not previously matching due to model missing a non-optional dps.
- Adds additional sensors seen on Catit feeder.
| - entity: sensor | ||
| icon: "mdi:paw" | ||
| name: Feed report | ||
| name: Last meal size |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Renaming entities after release causes problems for existing users, who may have automations referencing this.
| - id: 15 | ||
| name: sensor | ||
| type: integer | ||
| range: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Range is not necessary for a sensor.
| max: 12 | ||
| # Only present after feeding on Catit 43752 | ||
| optional: true | ||
| readonly: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Readonly is implied by the fact the entity is a sensor, so adding this is not necessary.
| icon: "mdi:paw" | ||
| name: Manual feed | ||
| category: config | ||
| name: Feed meal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Renaming after release causes problems
| - dps_val: 2 | ||
| value: true | ||
| - value: false | ||
| readonly: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Readonly is implied by the fact this is a binary_sensor
| - dps_val: 4 | ||
| value: true | ||
| - value: false | ||
| readonly: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implied
- entity renaming causes issues for existing users - readonly is not necessary to add on sensors - some tweaks to product details
|
@make-all Many thanks for reviewing, and making necessary changes. I did wonder whether the name changes were backwards compatible. I reasoned that the config was probably only used on setup, and hence wouldn't break existing (i.e. sensors setup with an older version would maintain their existing IDs). Sounds like I was wrong there - apologies for that. My motivation was to make the names a little more accurate. |