-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Support for Homebridge 2.0 #904
Comments
Did you already check if there are incompatibilities? Overall, I think the needed changes seem limited.. The only thing is that I do not yet know when I'll have time to go over them and test it. |
@itavero i can fork it and check and let you know my findings. |
@itavero - I review the code , you already using the correct classes and no where in the code i can find the old syntax that is obsoleted. So in some moment you can update package.json that "engines" is 2.0 ready. Instead of const Units = Characteristic.Units; you will need to use const Units = api.hap.Units; (z2m is using api.hap.Units ) |
@itavero i notice Koenkk is preparing Z2M v.2 , will that affect your plugin : Koenkk/zigbee2mqtt#24198 |
I'd have to double check the MQTT changes, but at first glance I don't see anything that would break it. |
Problem description
homebridge will be releasing version 2.0 which will have breaking changes to existing plugins
Suggested solution
Make it HB 2.0 compatible . @itavero What is the plan for supporting Homebridge 2.0 - will the plugin be impacted and will it be easy to make it compatible with 2.0.
Alternative solutions
No response
Additional context / information
https://github.com/homebridge/homebridge/wiki/Updating-To-Homebridge-v2
The text was updated successfully, but these errors were encountered: