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 Dmaker.fan.p39 #1784

Closed
wants to merge 4 commits into from

Conversation

Landrash
Copy link

@Landrash Landrash commented Jun 16, 2023

Adds support for fanmiot p39. Already supported by genericmiot but does not expose mode.
Compared to P33 there's a few changes.

Changes

Added to README and to fan_miot.py

Source

https://miot-spec.org/miot-spec-v2/instance?type=urn:miot-spec-v2:device:fan:0000A005:dmaker-p39:1

Function test

Tested each command and function with miiocli
miiocli fanmiot --ip 10.20.30.50 --token 000000000000000000000000 COMMAND

@rytilahti
Copy link
Owner

Hi @Landrash and thanks for the PR! Which mode is not supported (siid and piid?) when using the genericmiot? Looking at the linked spec file, the mode seems to be there.

@Landrash
Copy link
Author

I'm incorrect.
Sorry about that.

I'll close this. Should I make another pr just to the Readme adding it being supported by the genericmiiot?

@Landrash Landrash closed this Jun 18, 2023
@rytilahti
Copy link
Owner

Ah, thanks for letting me know! No need to add it to the readme as the genericmiot should cover all modern devices :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants