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

Improve Avatto ME81 thermostats #1742

Closed
wants to merge 1 commit into from

Conversation

erelke
Copy link
Contributor

@erelke erelke commented Mar 12, 2024

reflect hvac action on icon

@make-all
Copy link
Owner

Now that icon translations are supported, I don't want to add additional icon config like this. The default climate entity already has logic to handle hvac_action, and the "fix" is perhaps to remove any other icon customization that might be overriding it.

make-all added a commit that referenced this pull request Mar 14, 2024
The product info became confused somehow. When Avatto ME81 was
initially added as a compatible device, the ids somehow became switched.
But also the wrong name got assigned to the other id. Looking back at the
original report, it should have been Avatto WT200, not WT81.

Finally, there were detection issues for ME81, so another new device
report got filed for it later, and a seperate config made (even though it
could have handled by making a couple of dps optional).

Based on PR #1742
@make-all make-all closed this Mar 14, 2024
@make-all
Copy link
Owner

Manually merged the product name changes.

timlaing pushed a commit to timlaing/tuya-local that referenced this pull request Aug 8, 2024
The product info became confused somehow. When Avatto ME81 was
initially added as a compatible device, the ids somehow became switched.
But also the wrong name got assigned to the other id. Looking back at the
original report, it should have been Avatto WT200, not WT81.

Finally, there were detection issues for ME81, so another new device
report got filed for it later, and a seperate config made (even though it
could have handled by making a couple of dps optional).

Based on PR make-all#1742
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants