Skip to content

Updated AMP/Observer to include networking setup and OutdoorNav links #403

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

Merged
merged 9 commits into from
Jun 13, 2025

Conversation

jhiggins-cpr
Copy link
Collaborator

It probably makes sense for a combination of @rhysfaultless-cpr and @stephen-cpr to review this. At a minimum:

  • @rhysfaultless-cpr to confirm that the new content is consistent with the setup and flow of the existing AMP/Observer docs
  • @stephen-cpr to confirm that sequence of steps is correct and intuitive for our target customers

Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-403.dfy90wyu8dics.amplifyapp.com

@rhysfaultless-cpr
Copy link
Member

I added some inline comments for content and structure.

Separately, what is the goal of the Quick Start section?
This section seems to cause the reader to jump around the page, when compared to the chronological structure of Getting Started, Operating The Robot, Maintenance.
Could the goal be achieved by adding OutdoorNav content to Getting Started or Operating The Robot sections?

@jhiggins-cpr
Copy link
Collaborator Author

I added some inline comments for content and structure.

Separately, what is the goal of the Quick Start section? This section seems to cause the reader to jump around the page, when compared to the chronological structure of Getting Started, Operating The Robot, Maintenance. Could the goal be achieved by adding OutdoorNav content to Getting Started or Operating The Robot sections?

@rhysfaultless-cpr I restructured things based on our discussion. Let me know if you think this fits with your proposed structure.

@jhiggins-cpr jhiggins-cpr changed the title Added AMP/Observer Quick Start content Updated AMP/Observer to include networking setup and OutdoorNav links Jun 5, 2025
@rhysfaultless-cpr
Copy link
Member

@jhiggins-cpr I added inline comments. Overall the new layout looks good.

@jhiggins-cpr jhiggins-cpr merged commit 46bf34c into development Jun 13, 2025
2 checks passed
@jhiggins-cpr jhiggins-cpr deleted the amp_quickstart branch June 13, 2025 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants