-
Notifications
You must be signed in to change notification settings - Fork 16
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
Verification Request: @vandubois/homebridge-virtual-switch #752
Comments
🔴 The following checks failed:
🟢 The following checks passed:
If updating your |
/check |
🔴 The following checks failed:
🟢 The following checks passed:
If updating your |
/check |
🟢 The following checks passed:
🎉 All checks passed successfully, nice work! Your plugin and/or icon will now be manually reviewed by the Homebridge team. |
/check |
🟢 The following checks passed:
🎉 All checks passed successfully, nice work! Your plugin and/or icon will now be manually reviewed by the Homebridge team. |
|
Hi @Plankske i'm happy to verify the plugin, however i wondered if this situation could have a more helpful error message for the user. I have a minimal default config of this:
and this is leading to a (correctly caught) error of this:
I just wondered if this is a config-issue then the resolution could be made clearer to the user. |
Congratulations! Your plugin has been verified! You can now add one of the Verified by Homebridge badges to your plugin's README:
Your plugin is now also eligible to display a ❤️ Donate button on its tile in the Homebridge UI. See https://github.com/homebridge/homebridge/wiki/Donation-Links for instructions. If for any reason in the future you can no longer maintain your plugin, please consider transferring it to our unmaintained plugins repo. We can take ownership until another willing developer comes along. Don't forget to join the official Homebridge Discord server, where plugin developers can get tips and advice from other developers and the Homebridge project team in the #plugin-development channel! As a verified plugin, you can request a channel in the Discord server to discuss your plugin with users and other developers. Just ask in the #plugin-development channel. Thank you for your contribution to the Homebridge community.
|
Hi!
Thanks for the verification!
Made an update to alert the user about the error and how to resolve.
Keep up the good work! Very much appreciated!
Best Regards,
Dirk
From: Ben ***@***.***>
Sent: Sunday, September 22, 2024 5:28 AM
To: homebridge/verified ***@***.***>
Cc: Plankske ***@***.***>; Mention ***@***.***>
Subject: Re: [homebridge/verified] Verification Request: @vandubois/homebridge-virtual-switch (Issue #752)
Hi @Plankske<https://github.com/Plankske> i'm happy to verify the plugin, however i wondered if this situation could have a more helpful error message for the user.
I have a minimal default config of this:
{
"devices": [
{
"NormallyClosed": false,
"SwitchStayOn": false,
"Time": 5000,
"UseLogFile": false,
"LogFilePath": "/var/lib/homebridge/homebridge.log",
"EnableStartupDelay": false,
"StartupDelay": 10000,
"RememberState": false
}
],
"name": "HomebridgeVirtualSwitches",
"platform": "HomebridgeVirtualSwitches",
"_bridge": {
"username": "0E:E9:1B:29:78:5B",
"port": 37196
}
}
and this is leading to a (correctly caught) error of this:
[22/09/2024, 10:23:20] [HomebridgeVirtualSwitches] DEBUG: Executed didFinishLaunching callback
[22/09/2024, 10:23:20] [HomebridgeVirtualSwitches] DEBUG: Device config: {"SwitchStayOn":false,"Time":5000,"UseLogFile":false,"LogFilePath":"/var/lib/homebridge/homebridge.log","Keywords":[],"EnableStartupDelay":false,"StartupDelay":10000,"NormallyClosed":false,"RememberState":false}
[22/09/2024, 10:23:20] [HomebridgeVirtualSwitches] ERROR: Failed to load dependencies: TypeError: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received undefined
at Hash.update (node:internal/crypto/hash:115:11)
at Object.generate (/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/util/uuid.ts:11:11)
at HomebridgeVirtualSwitchesPlatform.discoverDevices ***@***.***/homebridge-virtual-switch/src/platform.ts:97:38)
at ***@***.***/homebridge-virtual-switch/src/platform.ts:47:14 {
code: 'ERR_INVALID_ARG_TYPE'
}
[22/09/2024, 10:23:20] Homebridge v2.0.0-beta.21 (HAP v1.1.1-beta.6) (HomebridgeVirtualSwitches) is running on port 37196.
I just wondered if this is a config-issue then the resolution could be made clearer to the user.
-
Reply to this email directly, view it on GitHub<#752 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/APYHGATA26UL23ZJN34NU7LZX2EQVAVCNFSM6AAAAABNHHSFQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRWGMZDQMJTGE>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>
|
Plugin Name
@vandubois/homebridge-virtual-switch
Link To GitHub Repo
https://github.com/Plankske/hb-virtual-switch
Plugin Icon (Optional)
No response
The plugin does not offer the same nor less functionality than that of any existing verified plugin.
🟢 Yes
The plugin successfully installs and does not start unless it is configured.
🟢 Yes
The plugin does not require the user to run Homebridge in a TTY or with non-standard startup parameters, even for initial configuration.
🟢 Yes
The plugin does not contain any analytics or calls that enable you to track the user.
🟢 Yes
If the plugin needs to write files to disk (cache, keys, etc.), it stores them inside the Homebridge storage directory.
🟢 Yes
The plugin does not throw unhandled exceptions, the plugin must catch and log its own errors.
🟢 Yes
More Information
The plugin is Homebridge 2.0 compliant and provides for virtual switches. The primary novelty is that the plugin can be used to monitor the Homebridge log file for user defined keywords (e.g. warning/alert messages that a plugin has lost API connection) . When they occur in the log file, a virtual switch is set (stateful or not, normally closed or not). These switches can then be used in advanced HomeKit automations (e.g. to send messages to the admin via Pushover, blow a horn, blink a light, etc.). This increases the practicality and reliability of Homebridge integrations of other (IoT) plugins greatly. Lastly, the plugin also allows for the creation of a variety of traditional virtual switches.
The text was updated successfully, but these errors were encountered: