-
Notifications
You must be signed in to change notification settings - Fork 41
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
DOCS-2321: Document fleet dashboard on manage fleet page #3631
DOCS-2321: Document fleet dashboard on manage fleet page #3631
Conversation
sguequierre
commented
Oct 25, 2024
- documents dashboard and all machines dashboard on manage machines how-to
- adds sentence documenting app locations page change on locations docs page
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.
Great - thank you. 2 comments then good to merge
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.
Actually - I just reviewed the description of DOCS-3005 (which you moved into reivew in progress so I assume you rolled that into this change). I don't think that work is fully done. The UI changed and we definitely have to change more guidance for users than we do here. I gave the manage-fleet page a cursory review (When you create a location you can't click on the location) and there are definitely other things that still need changing. Up to you whether you want to do that in this PR but it does need doing.
Co-authored-by: Naomi Pentrel <[email protected]>
docs/how-tos/manage-fleet.md
Outdated
A new location called `First Location` is automatically generated for you. | ||
|
||
Click on the location and use the **...** menu next to edit the location name to what it represents for your use case. | ||
Use the **...** menu next to edit the location name to what it represents for your use case. |
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.
did this bc it automatically takes you to it, not sure if new change
Co-authored-by: Naomi Pentrel <[email protected]>
Co-authored-by: Naomi Pentrel <[email protected]>
You can view a rendered version of the docs from this PR at https://docs-test.viam.dev/3631 |