Skip to content

Add asset validation docs page#163

Open
richturner wants to merge 9 commits into
mainfrom
richturner-patch-2
Open

Add asset validation docs page#163
richturner wants to merge 9 commits into
mainfrom
richturner-patch-2

Conversation

@richturner

Copy link
Copy Markdown
Member

No description provided.

@richturner richturner self-assigned this Jun 17, 2026
@richturner richturner requested a review from a team June 17, 2026 12:58
Comment thread docs/architecture/asset-validation.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Architecture documentation page describing asset validation, and reorders the Architecture docs sidebar positions to accommodate the new page.

Changes:

  • Add docs/architecture/asset-validation.md documenting the asset type model and constraint validation flow.
  • Update sidebar_position values across existing Architecture docs to insert the new page in the desired order.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
docs/architecture/security.md Adjust sidebar ordering to account for the new Asset Validation page.
docs/architecture/manager-endpoints-and-file-paths.md Adjust sidebar ordering to account for the new Asset Validation page.
docs/architecture/esp32-device.md Adjust sidebar ordering to account for the new Asset Validation page.
docs/architecture/asset-validation.md New documentation page describing asset validation model/flow and constraint sources.
docs/architecture/asset-location-tracking.md Adjust sidebar ordering to account for the new Asset Validation page.
docs/architecture/apps-and-consoles.md Adjust sidebar ordering to account for the new Asset Validation page.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/architecture/asset-validation.md Outdated
Comment thread docs/architecture/asset-validation.md Outdated
Comment thread docs/architecture/asset-validation.md Outdated
richturner and others added 4 commits June 17, 2026 16:23
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@richturner

Copy link
Copy Markdown
Member Author

Thanks for initial feedback. Ready for review @wborn

@wborn wborn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the build issues.

@richturner richturner requested a review from wborn June 19, 2026 10:13
@richturner richturner enabled auto-merge (squash) June 19, 2026 10:14
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.

3 participants