Add Blazor WASM telemetry docs and post-deployment observability guidance#455
Add Blazor WASM telemetry docs and post-deployment observability guidance#455
Conversation
Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
IEvangelist
left a comment
There was a problem hiding this comment.
This PR looks good so far, but let's double check it for accuracy by using the .github/skills/doc-tester/SKILL.md skill. Please also consider all new comments and feedback/suggestions. @copilot
|
Hey @copilot, please have a look at this comment: #455 (review) |
… improve URL guidance Co-authored-by: IEvangelist <7679720+IEvangelist@users.noreply.github.com>
Ran doc-tester validation using Playwright against the local dev server. Found and fixed three issues (commit
All internal anchor links ( Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
There was a problem hiding this comment.
Pull request overview
This PR addresses documentation gaps identified from Discord community feedback by adding comprehensive guidance for Blazor WebAssembly telemetry integration and post-deployment observability. The changes fill critical knowledge gaps around browser-based telemetry collection and production telemetry configuration that were frequently asked about in the community.
Changes:
- Added Blazor WASM telemetry integration section to
enable-browser-telemetry.mdxcovering OTEL config exposure, JS interop initialization, trace correlation, and authenticated proxy patterns - Created new
telemetry-after-deployment.mdxguide explaining development vs. production telemetry differences, Azure Monitor setup, troubleshooting missing logs, and dashboard access after deployment - Updated sidebar configuration to include the new telemetry after deployment page
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx |
New comprehensive guide covering production telemetry setup with Azure Monitor, troubleshooting missing logs, and accessing the deployed dashboard |
src/frontend/src/content/docs/dashboard/enable-browser-telemetry.mdx |
Added Blazor WebAssembly integration section with OTEL configuration API, component initialization, trace correlation, and authenticated proxy pattern |
src/frontend/config/sidebar/dashboard.topics.ts |
Added sidebar entry for the new telemetry after deployment page |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/frontend/src/content/docs/dashboard/enable-browser-telemetry.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
alistairmatthews
left a comment
There was a problem hiding this comment.
@copilot Please implement my suggested changes.
src/frontend/src/content/docs/dashboard/enable-browser-telemetry.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
src/frontend/src/content/docs/dashboard/telemetry-after-deployment.mdx
Outdated
Show resolved
Hide resolved
…hrasing improvements Co-authored-by: alistairmatthews <41286777+alistairmatthews@users.noreply.github.com>
Done in commit
|
alistairmatthews
left a comment
There was a problem hiding this comment.
Looks good. Ready to merge.


enable-browser-telemetry.mdxdashboard/telemetry-after-deployment.mdxdashboard.topics.tssidebar with translationsenable-browser-telemetry.mdx: Rephrase trace correlation paragraph (removed parenthetical)enable-browser-telemetry.mdx: Fix proxy code to useHttpRequestMessageinstead of mutatingclient.DefaultRequestHeaders(thread-safety for concurrent requests)telemetry-after-deployment.mdx: "vs." → "versus" in section headingtelemetry-after-deployment.mdx: RephrasedUseAzureMonitor()sentence for claritytelemetry-after-deployment.mdx: RephrasedAPPLICATIONINSIGHTS_CONNECTION_STRINGset-by-Aspire clausetelemetry-after-deployment.mdx: "when the OTEL endpoint is configured" → "when you configure the OTEL endpoint"telemetry-after-deployment.mdx: Removed parentheses from list items (if required, when using Azure Monitor)telemetry-after-deployment.mdx: "Container App" → "Container app" in Azure Portal nav pathtelemetry-after-deployment.mdx: "Telemetry sampled out" → "Telemetry sampling missed an event"telemetry-after-deployment.mdx: Expanded sampling explanation for claritytelemetry-after-deployment.mdx: Removed parentheses around "or similar"telemetry-after-deployment.mdx: Overview → Overview (bolded consistently)telemetry-after-deployment.mdx: "browser token auth" → "browser token authentication"Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.