Skip to content

Conversation

@flamingotter
Copy link
Contributor

This is a hisense Window air conditioner. tested and working within my HA environment

- property: t_beep
disable: true
- property: t_eco
disable: true
Copy link
Owner

Choose a reason for hiding this comment

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

Does the device expose any of the disabled properties?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not that I was able to identify. This is a small room window AC unit with minimal features.

Copy link
Owner

Choose a reason for hiding this comment

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

So can the disabled properties just be deleted from the mapping file? Or are they included in the statusList for the device, but just don't have any effect?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

can be deleted. I was just following how the other files were structured. I've commented them out of my local config and reloaded system without error.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

commited change

@oyvindwe
Copy link
Owner

Thank you for your contribution ❤️

Just left a question in the review.

@oyvindwe oyvindwe changed the title Create 008-304.yaml Added support for device 008-304 (HiSense Window air conditioner) Oct 23, 2024
@oyvindwe
Copy link
Owner

oyvindwe commented Nov 3, 2024

I think it's cleaner to just remove the properties, but I'm happy to merge in any case.

Thank you for your contribution!

@oyvindwe oyvindwe merged commit b0fbce4 into oyvindwe:main Nov 3, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants