Skip to content
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: homebridge-alphaess #720

Closed
zerwuff opened this issue Apr 4, 2024 · 32 comments
Closed

Verification Request: homebridge-alphaess #720

zerwuff opened this issue Apr 4, 2024 · 32 comments
Labels
verified use when a plugin meets the criteria - adds the verified badge text

Comments

@zerwuff
Copy link

zerwuff commented Apr 4, 2024

Plugin Name

homebridge-alphaess

Link To GitHub Repo

https://github.com/zerwuff/homebridge-alphaess

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

Homebridge Plugin for Integration Alpha ESS Energy Storage Systems into the Home App.

This is done by integrating the alpha ess open api (REST) into the homebridge platform

@zerwuff zerwuff added the pending the label given to a new verification/icon request label Apr 4, 2024
Copy link

github-actions bot commented Apr 4, 2024

🔴 The following pre-checks failed:

  • Package JSON: more keywords apart from 'homebridge-plugin' should exist
  • GitHub Repo: should have issues enabled
  • GitHub Repo: should contain releases
  • Config Schema JSON: does not contain a name schema property

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Package JSON: repository.url property exists
  • Package JSON: bugs.url exists
  • Package JSON: 'preinstall' in scripts is not present
  • Package JSON: 'install' in scripts is not present
  • Package JSON: 'postinstall' in scripts is not present
  • Package JSON: engines.node property is compatible with Node 18
  • Package JSON: engines.node property is compatible with Node 20
  • Package JSON: engines.homebridge property is compatible with Homebridge 1.7.0
  • Package JSON: initializer function found
  • GitHub Repo: repository is public
  • GitHub Repo: repository is not archived
  • NPM Package: has not been deprecated
  • Config Schema JSON: exists and is valid JSON
  • Config Schema JSON: contains a valid pluginAlias
  • Config Schema JSON: the pluginType is set to 'platform'
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.

@github-actions github-actions bot added awaiting-changes use after review has started - awaiting user to make changes to plugin and removed pending the label given to a new verification/icon request labels Apr 4, 2024
@zerwuff
Copy link
Author

zerwuff commented Apr 4, 2024

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed awaiting-changes use after review has started - awaiting user to make changes to plugin labels Apr 4, 2024
Copy link

github-actions bot commented Apr 4, 2024

🔴 The following pre-checks failed:

  • GitHub Repo: should have issues enabled
  • GitHub Repo: should contain releases
  • Config Schema JSON: does not contain a name schema property

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Package JSON: repository.url property exists
  • Package JSON: bugs.url exists
  • Package JSON: keywords exist and contain 'homebridge-plugin'
  • Package JSON: 'preinstall' in scripts is not present
  • Package JSON: 'install' in scripts is not present
  • Package JSON: 'postinstall' in scripts is not present
  • Package JSON: engines.node property is compatible with Node 18
  • Package JSON: engines.node property is compatible with Node 20
  • Package JSON: engines.homebridge property is compatible with Homebridge 1.7.0
  • Package JSON: initializer function found
  • GitHub Repo: repository is public
  • GitHub Repo: repository is not archived
  • NPM Package: has not been deprecated
  • Config Schema JSON: exists and is valid JSON
  • Config Schema JSON: contains a valid pluginAlias
  • Config Schema JSON: the pluginType is set to 'platform'
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.Note that the package.json and config.schema.json files are read from the latest NPM release, not the GitHub repo.

@github-actions github-actions bot added awaiting-changes use after review has started - awaiting user to make changes to plugin and removed pending the label given to a new verification/icon request labels Apr 4, 2024
@bwp91
Copy link
Contributor

bwp91 commented Apr 4, 2024

maybe there is something not quite right with the github repo checks... will check later

@zerwuff
Copy link
Author

zerwuff commented Apr 4, 2024

@bwp91 thanks for feedback, im a bit confused since most of the things area resolved. hmm.

@zerwuff
Copy link
Author

zerwuff commented Apr 4, 2024

/check

@github-actions github-actions bot added the pending the label given to a new verification/icon request label Apr 4, 2024
Copy link

github-actions bot commented Apr 4, 2024

🔴 The following pre-checks failed:

  • GitHub Repo: should have issues enabled
  • GitHub Repo: should contain releases
  • Config Schema JSON: does not contain a name schema property

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Package JSON: repository.url property exists
  • Package JSON: bugs.url exists
  • Package JSON: keywords exist and contain 'homebridge-plugin'
  • Package JSON: 'preinstall' in scripts is not present
  • Package JSON: 'install' in scripts is not present
  • Package JSON: 'postinstall' in scripts is not present
  • Package JSON: engines.node property is compatible with Node 18
  • Package JSON: engines.node property is compatible with Node 20
  • Package JSON: engines.homebridge property is compatible with Homebridge 1.7.0
  • Package JSON: initializer function found
  • GitHub Repo: repository is public
  • GitHub Repo: repository is not archived
  • NPM Package: has not been deprecated
  • Config Schema JSON: exists and is valid JSON
  • Config Schema JSON: contains a valid pluginAlias
  • Config Schema JSON: the pluginType is set to 'platform'
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.Note that the package.json and config.schema.json files are read from the latest NPM release, not the GitHub repo.

@github-actions github-actions bot removed the pending the label given to a new verification/icon request label Apr 4, 2024
@bwp91
Copy link
Contributor

bwp91 commented Apr 4, 2024

The github issues are failing because your package json file contains a different repo URL:

  "repository": {
    "type": "git",
    "url": "https://github.com/2rw0lff/homebridge-alphaess.git"
  },
  "bugs": {
    "url": "https://github.com/2rw0lff/homebridge-alphaess/issues"
  },

whereas in your ticket you specified https://github.com/zerwuff/homebridge-alphaess

@zerwuff
Copy link
Author

zerwuff commented Apr 5, 2024

@bwp91 thanks

@zerwuff
Copy link
Author

zerwuff commented Apr 5, 2024

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed awaiting-changes use after review has started - awaiting user to make changes to plugin labels Apr 5, 2024
Copy link

github-actions bot commented Apr 5, 2024

🔴 The following pre-checks failed:

  • GitHub Repo: should have issues enabled
  • GitHub Repo: should contain releases
  • Config Schema JSON: does not contain a name schema property

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Package JSON: repository.url property exists
  • Package JSON: bugs.url exists
  • Package JSON: keywords exist and contain 'homebridge-plugin'
  • Package JSON: 'preinstall' in scripts is not present
  • Package JSON: 'install' in scripts is not present
  • Package JSON: 'postinstall' in scripts is not present
  • Package JSON: engines.node property is compatible with Node 18
  • Package JSON: engines.node property is compatible with Node 20
  • Package JSON: engines.homebridge property is compatible with Homebridge 1.7.0
  • Package JSON: initializer function found
  • GitHub Repo: repository is public
  • GitHub Repo: repository is not archived
  • NPM Package: has not been deprecated
  • Config Schema JSON: exists and is valid JSON
  • Config Schema JSON: contains a valid pluginAlias
  • Config Schema JSON: the pluginType is set to 'platform'
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.Note that the package.json and config.schema.json files are read from the latest NPM release, not the GitHub repo.

@github-actions github-actions bot added awaiting-changes use after review has started - awaiting user to make changes to plugin and removed pending the label given to a new verification/icon request labels Apr 5, 2024
@zerwuff
Copy link
Author

zerwuff commented Apr 5, 2024

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed awaiting-changes use after review has started - awaiting user to make changes to plugin labels Apr 5, 2024
@bwp91
Copy link
Contributor

bwp91 commented Apr 5, 2024

/check

Copy link

github-actions bot commented Apr 5, 2024

🔴 The following pre-checks failed:

  • Config Schema JSON: does not contain a name schema property

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Package JSON: repository.url property exists
  • Package JSON: bugs.url exists
  • Package JSON: keywords exist and contain 'homebridge-plugin'
  • Package JSON: 'preinstall' in scripts is not present
  • Package JSON: 'install' in scripts is not present
  • Package JSON: 'postinstall' in scripts is not present
  • Package JSON: engines.node property is compatible with Node 18
  • Package JSON: engines.node property is compatible with Node 20
  • Package JSON: engines.homebridge property is compatible with Homebridge 1.7.0
  • Package JSON: initializer function found
  • GitHub Repo: repository is public
  • GitHub Repo: repository is not archived
  • GitHub Repo: issues are enabled
  • GitHub Repo: contains releases
  • NPM Package: has not been deprecated
  • Config Schema JSON: exists and is valid JSON
  • Config Schema JSON: contains a valid pluginAlias
  • Config Schema JSON: the pluginType is set to 'platform'
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.Note that the package.json and config.schema.json files are read from the latest NPM release, not the GitHub repo.

@github-actions github-actions bot added the awaiting-changes use after review has started - awaiting user to make changes to plugin label Apr 5, 2024
@zerwuff
Copy link
Author

zerwuff commented Apr 6, 2024

hi @bwp91 still no success on it zerwuff/homebridge-alphaess@f18e8a6 what did i miss ?

@bwp91
Copy link
Contributor

bwp91 commented Apr 6, 2024

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed awaiting-changes use after review has started - awaiting user to make changes to plugin labels Apr 6, 2024
Copy link

github-actions bot commented Apr 6, 2024

🔴 The following pre-checks failed:

  • Config Schema JSON: does not contain a name schema property

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Package JSON: repository.url property exists
  • Package JSON: bugs.url exists
  • Package JSON: keywords exist and contain 'homebridge-plugin'
  • Package JSON: 'preinstall' in scripts is not present
  • Package JSON: 'install' in scripts is not present
  • Package JSON: 'postinstall' in scripts is not present
  • Package JSON: engines.node property is compatible with Node 18
  • Package JSON: engines.node property is compatible with Node 20
  • Package JSON: engines.homebridge property is compatible with Homebridge 1.7.0
  • Package JSON: initializer function found
  • GitHub Repo: repository is public
  • GitHub Repo: repository is not archived
  • GitHub Repo: issues are enabled
  • GitHub Repo: contains releases
  • NPM Package: has not been deprecated
  • Config Schema JSON: exists and is valid JSON
  • Config Schema JSON: contains a valid pluginAlias
  • Config Schema JSON: the pluginType is set to 'platform'
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.

If updating your package.json and config.schema.json files, don't forget to publish a new version to NPM.

@github-actions github-actions bot added awaiting-changes use after review has started - awaiting user to make changes to plugin and removed pending the label given to a new verification/icon request labels Apr 6, 2024
@bwp91
Copy link
Contributor

bwp91 commented Apr 6, 2024

the latest npm release of your plugin does not include the change

https://www.npmjs.com/package/homebridge-alphaess?activeTab=code

@zerwuff
Copy link
Author

zerwuff commented Apr 7, 2024

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed awaiting-changes use after review has started - awaiting user to make changes to plugin labels Apr 7, 2024
Copy link

github-actions bot commented Apr 7, 2024

🔴 The following pre-checks failed:

  • Config Schema JSON: does not contain a name schema property

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Package JSON: repository.url property exists
  • Package JSON: bugs.url exists
  • Package JSON: keywords exist and contain 'homebridge-plugin'
  • Package JSON: 'preinstall' in scripts is not present
  • Package JSON: 'install' in scripts is not present
  • Package JSON: 'postinstall' in scripts is not present
  • Package JSON: engines.node property is compatible with Node 18
  • Package JSON: engines.node property is compatible with Node 20
  • Package JSON: engines.homebridge property is compatible with Homebridge 1.7.0
  • Package JSON: initializer function found
  • GitHub Repo: repository is public
  • GitHub Repo: repository is not archived
  • GitHub Repo: issues are enabled
  • GitHub Repo: contains releases
  • NPM Package: has not been deprecated
  • Config Schema JSON: exists and is valid JSON
  • Config Schema JSON: contains a valid pluginAlias
  • Config Schema JSON: the pluginType is set to 'platform'
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

⚠️ Please action these failures and then comment /check to run the checks again. Let us know if you need any help.

If updating your package.json and config.schema.json files, don't forget to publish a new version to NPM.

@github-actions github-actions bot added awaiting-changes use after review has started - awaiting user to make changes to plugin and removed pending the label given to a new verification/icon request labels Apr 7, 2024
@zerwuff
Copy link
Author

zerwuff commented Apr 7, 2024

hmm @bwp91 just created a new npm release and github release including the last commit. still failing, any ideas ?

@zerwuff
Copy link
Author

zerwuff commented Apr 7, 2024

/check

@github-actions github-actions bot added pending the label given to a new verification/icon request and removed awaiting-changes use after review has started - awaiting user to make changes to plugin labels Apr 7, 2024
Copy link

github-actions bot commented Apr 7, 2024

🟢 The following pre-checks passed:

  • Installation: successfully installed
  • Package JSON: repository.url property exists
  • Package JSON: bugs.url exists
  • Package JSON: keywords exist and contain 'homebridge-plugin'
  • Package JSON: 'preinstall' in scripts is not present
  • Package JSON: 'install' in scripts is not present
  • Package JSON: 'postinstall' in scripts is not present
  • Package JSON: engines.node property is compatible with Node 18
  • Package JSON: engines.node property is compatible with Node 20
  • Package JSON: engines.homebridge property is compatible with Homebridge 1.7.0
  • Package JSON: initializer function found
  • GitHub Repo: repository is public
  • GitHub Repo: repository is not archived
  • GitHub Repo: issues are enabled
  • GitHub Repo: contains releases
  • NPM Package: has not been deprecated
  • Config Schema JSON: exists and is valid JSON
  • Config Schema JSON: contains a valid pluginAlias
  • Config Schema JSON: the pluginType is set to 'platform'
  • Config Schema JSON: contains a name schema property
  • Dependencies: homebridge was not installed as a dependency
  • Dependencies: hap-nodejs was not installed as a dependency

🎉 All pre-checks passed successfully, nice work! Your plugin and/or icon will now be manually reviewed by the Homebridge team.

@bwp91 bwp91 added currently-reviewing use for starting a review - adds a comment with the verification list with empty checkboxes and removed pending the label given to a new verification/icon request labels Apr 7, 2024
Copy link

github-actions bot commented Apr 7, 2024

  • General
    • The plugin must be of type dynamic platform.
    • The plugin must not offer the same nor less functionality than that of any existing verified plugin.
  • Repo
    • The plugin must be published to NPM and the source code available on a GitHub repository, with issues enabled.
    • A GitHub release should be created for every new version of your plugin, with release notes.
  • Environment
    • The plugin must run on all supported LTS versions of Node.js, at the time of writing this is Node v18 and v20.
    • The plugin must successfully install and not start unless it is configured.
    • The plugin must not execute post-install scripts that modify the users' system in any way.
    • The plugin must not require the user to run Homebridge in a TTY or with non-standard startup parameters, even for initial configuration.
  • Codebase
    • The plugin must implement the Homebridge Plugin Settings GUI.
    • The plugin must not contain any analytics or calls that enable you to track the user.
    • If the plugin needs to write files to disk (cache, keys, etc.), it must store them inside the Homebridge storage directory.
    • The plugin must not throw unhandled exceptions, the plugin must catch and log its own errors.

@bwp91 bwp91 added verified use when a plugin meets the criteria - adds the verified badge text and removed currently-reviewing use for starting a review - adds a comment with the verification list with empty checkboxes labels Apr 7, 2024
Copy link

github-actions bot commented Apr 7, 2024

Congratulations! Your plugin has been verified!

You can now add one of the Verified by Homebridge badges to your plugin's README:

verified-by-homebridge

[![verified-by-homebridge](https://badgen.net/badge/homebridge/verified/purple)](https://github.com/homebridge/homebridge/wiki/Verified-Plugins)

verified-by-homebridge

[![verified-by-homebridge](https://img.shields.io/badge/homebridge-verified-blueviolet?color=%23491F59&style=for-the-badge&logoColor=%23FFFFFF&logo=homebridge)](https://github.com/homebridge/homebridge/wiki/Verified-Plugins)

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.

  • The Homebridge Team

bwp91 added a commit that referenced this issue Apr 7, 2024
@bwp91
Copy link
Contributor

bwp91 commented Apr 7, 2024

Hi @zerwuff all good!

I was just looking at your package dependencies and wondered if there was any that could be moved to devDependencies?

In particular I was thinking about jest-mock-server, but any others that aren't needed that can be removed, or can be moved to dev dependencies.

No problem of course if no changes are needed!

@zerwuff
Copy link
Author

zerwuff commented Apr 7, 2024

hi @bwp91 thanks for the review & feedback. i did a garbage collection on package.json and did remove the jest-mock-server and some other unused stuff. thanks for looking into it!

@zerwuff zerwuff closed this as completed Apr 7, 2024
@zerwuff
Copy link
Author

zerwuff commented Apr 7, 2024

thanks for support

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified use when a plugin meets the criteria - adds the verified badge text
Projects
None yet
Development

No branches or pull requests

2 participants