-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
Can't configure the plugin. #298
Comments
I suspect that this is due to a change in Node.js 22, probably nodejs/node#51622. It looks like it should be a simple fix to adopt the new syntax for Node.js 22, and that new syntax appears to have been backported to Node.js 18 and 20… but I will need to run some tests to be sure, and will probably need to increase the minimum compatible version numbers on those branches, which will require some users to upgrade. |
In the meantime, should I downgrade to node 20?
… On Sep 16, 2024, at 3:34 PM, Alexander Thoukydides ***@***.***> wrote:
I suspect that this is due to a change in Node.js 22, probably nodejs/node#51622 <nodejs/node#51622>. It looks like it should be a simple fix to adopt the new syntax for Node.js 22, and that new syntax appears to have been backported to Node.js 18 and 20… but I will need to run some tests to be sure, and will probably need to increase the minimum compatible version numbers on those branches, which will require some users to upgrade.
—
Reply to this email directly, view it on GitHub <#298 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AENJQIGCVGIA27JY4EON5R3ZW4XD5AVCNFSM6AAAAABOJ5LJEGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJTG42TINJUHA>.
You are receiving this because you authored the thread.
|
I did downgrade to node 20.17 and it works fine.
… On Sep 16, 2024, at 3:34 PM, Alexander Thoukydides ***@***.***> wrote:
I suspect that this is due to a change in Node.js 22, probably nodejs/node#51622 <nodejs/node#51622>. It looks like it should be a simple fix to adopt the new syntax for Node.js 22, and that new syntax appears to have been backported to Node.js 18 and 20… but I will need to run some tests to be sure, and will probably need to increase the minimum compatible version numbers on those branches, which will require some users to upgrade.
—
Reply to this email directly, view it on GitHub <#298 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AENJQIGCVGIA27JY4EON5R3ZW4XD5AVCNFSM6AAAAABOJ5LJEGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJTG42TINJUHA>.
You are receiving this because you authored the thread.
|
Homebridge officially only supports Node.js LTS releases - which are currently 18 and 20 - so downgrading is a good workaround. However, Node.js 22 is expected to become an LTS release next month (October 2024) so I do need to fix this. It looks like the fix will require a minimum of Node.js 18.20.0. |
Should be fixed in v1.0.4. |
Description of Issue
Can't configure the plugin. I tried using the UI, also tried manually editing the JSON.
ERROR LOADING PLUGIN homebridge-homeconnect SyntaxError Unexpected identifier assert
Expected Behaviour
Get to step where I get QR code for child bridge
Steps to Reproduce
Used UI or manual install - no luck
Plugin Version
1.0.3
Environment
Home Connect Appliance(s)
Bosch frig
HomeKit App(s)
Apple Home
Diagnostic Checks
Log File
Configuration
Additional Information
No response
The text was updated successfully, but these errors were encountered: