-
-
Notifications
You must be signed in to change notification settings - Fork 700
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
Add-on suggestion finder revised schema #2174
Conversation
Signed-off-by: Andrew Fiddian-Green <[email protected]>
@mherwege for info.. |
✅ Thanks for your pull request to the openHAB documentation! The result can be previewed at the URL below (this comment and the preview will be updated if you add more commits).
To edit notification comments on pull requests, go to your Netlify site configuration. |
You should probably add to the documentation that different add-on discovery finders may need different discovery-parameters and refer to the specific finders javadoc for it. As an example, you could cite mDNS. |
Signed-off-by: Andrew Fiddian-Green <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@kaikreuzer could you merge this one too please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
See openhab/openhab-core#3924
This adjusts the documentation of the revised schema to match that in the above-mentioned PR.
Signed-off-by: Andrew Fiddian-Green [email protected]