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

Fixes issues/146 and issues/180 #194

Merged
merged 4 commits into from
May 18, 2020

Conversation

bedrin
Copy link
Collaborator

@bedrin bedrin commented May 16, 2020

Copy link
Member

@afharo afharo left a comment

Choose a reason for hiding this comment

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

It looks good!
I'd rather fix the success validation in here: https://github.com/homebridge-xiaomi-roborock-vacuum/miio/blob/master/lib/devices/vacuum.js#L11-L18 instead of moving every command from the miio library to this module (it will make it harder to support other models like #150). But we can tackle that later on.

@afharo afharo merged commit 59b2d30 into homebridge-xiaomi-roborock-vacuum:next May 18, 2020
@afharo
Copy link
Member

afharo commented May 18, 2020

Published as [email protected]

@bedrin
Copy link
Collaborator Author

bedrin commented May 18, 2020

I'd rather fix the success validation in here: https://github.com/homebridge-xiaomi-roborock-vacuum/miio/blob/master/lib/devices/vacuum.js#L11-L18 instead of moving every command from the miio library to this module (it will make it harder to support other models like #150). But we can tackle that later on.

I've raised a PR against miio but I think that project is discontinued - latest commit was over 2 years ago :(
aholstenson/miio#283

@afharo
Copy link
Member

afharo commented May 18, 2020

Yeah, you should push it to the fork under this organisation: https://github.com/homebridge-xiaomi-roborock-vacuum/miio
We are publishing that as miio-nicoh88 (you'll see that in the package.json in this module) because the original dev never answered nor reviewed our PRs.

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