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

Add support for Petoneer Fresco EzGo pet fountain (WF005) #2880

Merged
merged 2 commits into from
Feb 5, 2025

Conversation

K900
Copy link
Contributor

@K900 K900 commented Jan 30, 2025

Fixes #2878, except for scheduling, which will require some custom code.

make-all added a commit that referenced this pull request Feb 4, 2025
- should be translation_key, lights don't support classes

Noticed PR #2880 has copied it from somewhere, and found 3 occurrences.
- dps_val: smart
value: geyser
- entity: sensor
name: Filter cleaning in
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally I would use a name of "Filter remaining". It will make later translations easier if it is consistent with other configs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copied this from the Fresco Mini config, that might need updating too, as I now see other places are more consistent.

- dps_val: 1
value: true
- entity: binary_sensor
name: Error
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the name to let the class translations show through.

Generally also, I would include a "fault_code" attribute, with the raw value of dp 23.
I'd also suggest separating routine things out from the maintenance issues to their own binary_sensor (dps_val: 1 especially, there is a tank_empty translation key defined for such binary_sensors).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's kind of weird, this mostly overlaps water_level, which is why I kept it like that.

Fixes make-all#2878, except for scheduling, which will require some custom code.
@make-all make-all merged commit 6696981 into make-all:main Feb 5, 2025
4 checks passed
make-all added a commit that referenced this pull request Feb 5, 2025
- add model_id from PR

Issue #2878
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Request support for Petoneer WF005 pet fountain
2 participants