-
Notifications
You must be signed in to change notification settings - Fork 10
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
chore(docs) Address documentation updates that need review #7
Comments
"Add info on nearby networks: https://openeew.com/docs/contributing/" Not sure if this is required. |
"Add test plan info back in if needed: https://openeew.com/docs/contributing/" This was added in the github docs of the scripts directly |
"Add metadata for the Python library https://pypi.org/project/openeew/" Could you give an example? @krook |
@luisera It looks like this was broken, then resolved, but now it shows "The author of this package has not provided a project description" on https://pypi.org/project/openeew/#description |
@krook I lifted some language from the ClusterDuck Contributing file with regards to DCO. |
Related, particularly on the DCO bot: openeew/openeew#9 |
Any ideas on the open checklist item "Fix the 'access the credentials here' link on https://openeew.com/docs/deploy-detection-nodered"? I can't figure out how to run the first Node-RED example without a broker to connect to. Maybe I'm missing pointers to some other documentation on the brokers. Thanks! |
@djhunt-ibm this might have the missing piece. If it does, we should sync the repo with the tutorial: https://developer.ibm.com/events/call-for-code-live-build-node-red-dashboards-using-earthquake-sensor-data/ FYI @johnwalicki |
I did not see it either of those places, which led me here. The sample code for example one has a blank host name for the MQTT broker, and I wasn't sure if there's a public feed that we can attach to or if we have to simulate our own feed. Thanks! |
There are a few places in the docs that need a review and a decision on what change needs to be made. I've made obvious changes in this related pull request.
Populate these blank pages
The text was updated successfully, but these errors were encountered: