-
Notifications
You must be signed in to change notification settings - Fork 1
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
[RSDK-8760] - README improvements #21
Conversation
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.
One little nit, but otherwise lgtm
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.
I don't see it on the registry yet? Should we run viam module update
after merge? Also should we get someone from docs team to look at the README updates?
Cutting release once this PR is in.
Probably a good idea! |
README.md
Outdated
] | ||
} | ||
``` | ||
|
||
Make sure to configure a [Data Manager Service](https://docs.viam.com/services/data/cloud-sync/) to uplaod video files to the cloud. | ||
### Configure a Data Manager Service |
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.
Would suggest moving this up to above configure a video-store
section in a section called ## Requirements and also adding the instructions to configure through the UI:
Navigate to the [**CONFIGURE** tab](https://docs.viam.com/configure/) of your [machine](https://docs.viam.com/fleet/machines/) in [the Viam app](https://app.viam.com/). [Add data-management to your machine](https://docs.viam.com/configure/#services).
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.
Added requirements section.
Co-authored-by: Sierra Guequierre <sguequierre@gmail.com>
Co-authored-by: Sierra Guequierre <sguequierre@gmail.com>
Co-authored-by: Sierra Guequierre <sguequierre@gmail.com>
Co-authored-by: Sierra Guequierre <sguequierre@gmail.com>
Description