-
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
homebridge-spruce-irrigation #529
Comments
✅ Pre-checks completed successfully. |
If I configure an auth token that is invalid, this crashes the plugin and sends homebridge into a restart loop: Config:
Result:
|
This is not part of the verification, but I would like to recommend a Something like: "schema": {
"type": "object",
"properties": {
"name": {
"title": "Plugin name as shown in Homebridge Log",
"default": "SpruceIrrigation"
},
... |
Once I add that to the config, where do I tell the logger to use it when logging? |
You don’t need to do anything extra, it should use it automatically |
Ok, v1.1.1 just published should address both these items. Thanks |
/check |
✅ Pre-checks completed successfully. |
Plugin is no longer crashing homebridge with an invalid token - so good! I noticed that I still get these constant log entries when the token is invalid - so I assume some interval is running.
is this interval necessary when the token is invalid? Was just a thought of mine - is out of the scope for verification. |
Congratulations! Your plugin has been verified. You can now add the Verified by Homebridge badge to your plugin's README:
Your plugin is now also eligible to display a ❤️ Donate button on it's tile in the Homebridge UI. See https://github.com/oznu/homebridge-config-ui-x/wiki/Developers:-Donation-Links for instructions. Thank you for your contribution to the Homebridge Community. |
If the token was ever valid, the interval will keep updating HomeKit with last known values. If it was never valid, then yeah it's a bit useless. I'll make a note to look into it for the next version. Thanks for the help getting me verified. |
Link To GitHub Repo
https://github.com/waltmoorhouse/homebridge-spruce-irrigation/
Link To NPM Package
https://www.npmjs.com/package/homebridge-spruce-irrigation
The text was updated successfully, but these errors were encountered: