-
Notifications
You must be signed in to change notification settings - Fork 17
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: @theproductroadmap/homebridge-adjustable-timer #775
Comments
🔴 The following checks failed:
If updating your |
/check |
🔴 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 failed:
🟢 The following checks passed:
If updating your |
/check |
🔴 The following checks failed:
🟢 The following checks passed:
If updating your |
/check |
Hello, apologies for a lot of checks I'm just unsure about where this issue is coming from as the name property is picked up in HB when deployed and keywords are correctly picked up by npm. Any ideas? would really appreciate it! :) Thanks |
/check |
🔴 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. |
/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. |
With a default config of the following (I just pressed save on the plugin config window without changing anything) {
"name": "AdjustableDummyTimerPlatform",
"timers": [
{
"brightness": 0,
"delayUnit": "m",
"sensor": "off",
"pausable": false,
"disableLogging": false
}
],
"platform": "AdjustableDummyTimerPlatform"
} i am getting a repeated error in the log
I'm guessing it is because a 'required' value is missing from my config, but the plugin should check for the existence of the required fields, and if not present, then log a warning and return (rather than it causing an uncaught error as above) |
/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. These checks were run against v2.1.7 of the plugin. |
Plugin Name
@theproductroadmap/homebridge-adjustable-timer
Link To GitHub Repo
https://github.com/mksvrcek/homebridge-adjustable-timer
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
No response
The text was updated successfully, but these errors were encountered: