Skip to content

Initial Fan Addition#263

Draft
DellanX wants to merge 5 commits into
wez:mainfrom
DellanX:feature/fan-control
Draft

Initial Fan Addition#263
DellanX wants to merge 5 commits into
wez:mainfrom
DellanX:feature/fan-control

Conversation

@DellanX
Copy link
Copy Markdown

@DellanX DellanX commented Aug 10, 2024

I started to work on the code for the issue I created for #262, basing the logic off of the humidifier implementation.

It still needs some work integrating the MQTT class into your framework. I'm having issues getting my environment setup, but once I got it, I should be able to tackle getting a working fan device conversion.

@DellanX
Copy link
Copy Markdown
Author

DellanX commented Aug 11, 2024

This is about as far as I am willing to take the implementation, as I don't the time to learn how to add tests, or how to setup the build process so that I can prototype it.

@OHB
Copy link
Copy Markdown

OHB commented Aug 3, 2025

This is about as far as I am willing to take the implementation, as I don't the time to learn how to add tests, or how to setup the build process so that I can prototype it.

Hi @DellanX, I was wondering where you ended up with this? If it adds functionality for fans, I'm thinking I could fork and apply this PR...that is, unless you have a later version or any thoughts or advice? Cheers!

@DellanX
Copy link
Copy Markdown
Author

DellanX commented Aug 4, 2025

I wasnt able to get the addon to build to validate my changes. Also, I think there are a few typos in there.

The main thing is, to register to a fan mqtt entity the different mqtt entities to read/update to implement a fans different features.

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