add doc around dashboard telemetry #2846
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
IEvangelist
reviewed
Mar 21, 2025
There was a problem hiding this comment.
We will need a TOC entry for this. I updated the PR to target the release-9.2 branch. I left some early comments to help shape the content here, this is a great start @adamint
There was a problem hiding this comment.
Can you get an even larger image? If so, that should be used as the lightbox. Right now, using the same image in this situation doesn't make sense as when you select it, it's not larger than the one currently rendered. Also, we've been defaulting to using the dark theme for docs.
There was a problem hiding this comment.
I can, but on second thought, I'm not sure this image is necessary to include at all. What's your opinion? In the meantime, I removed it.
| ms.date: 03/05/2025 | ||
| --- | ||
|
|
||
| # .NET Aspire dashboard telemetry |
There was a problem hiding this comment.
What do you think about this for the title instead?
Suggested change
| # .NET Aspire dashboard telemetry | |
| # Microsoft collected dashboard telemetry |
".NET Aspire telemetry" is now officially an oxymoron; in that it has two contradictory meanings. Before 9.2, the dashboard was all about display resource telemetry, obviously we all know that, but now, we're saying ".NET Aspire telemetry" in this article is about the data Microsoft collects to improve the dashboard experience.
Or perhaps something like ".NET Aspire dashboard usage telemetry"?
There was a problem hiding this comment.
Microsoft-collected dashboard telemetry works for me!
|
|
||
| Protecting your privacy is important to us. If you suspect that telemetry is collecting sensitive data or the data is being insecurely or inappropriately handled, file an issue in the [dotnet/aspire](https://github.com/dotnet/aspire) repository for investigation. | ||
|
|
||
| The Aspire dashboard does not collect telemetry on Visual Studio versions `< 17.14` or C# Dev Kit versions `< [VSC RELEASE VERSION`. It collects the following data: |
There was a problem hiding this comment.
Suggested change
| The Aspire dashboard does not collect telemetry on Visual Studio versions `< 17.14` or C# Dev Kit versions `< [VSC RELEASE VERSION`. It collects the following data: | |
| The .NET Aspire dashboard doesn't collect telemetry on Visual Studio versions `< 17.14` or C# Dev Kit versions `< [VSC RELEASE VERSION]`. It collects the following data: |
IEvangelist
reviewed
Mar 26, 2025
docs/fundamentals/dashboard/microsoft-collected-dashboard-telemetry.md
Outdated
Show resolved
Hide resolved
adamint
changed the title
IEvangelist
approved these changes
May 8, 2025
IEvangelist
added a commit
that referenced
this pull request
May 9, 2025
* wip * wip * update * add image * Revision based on suggestions, add to TOC * Apply suggestions from code review * Clarify command/resource type names not sent --------- Co-authored-by: Adam Ratzman <[email protected]> Co-authored-by: David Pine <[email protected]>
IEvangelist
added a commit
that referenced
this pull request
May 19, 2025
* Update existing content to address changes mentioned in #3078. * Add conditional nuget.config for release branch * Add breaking change to fix #3326 * Added a step to print the trigger context * Fix default expanded state * Remove text * Tweak TOC * Fix MD warings * Added SKU changes, fixed #3144 * Remove workflows and let main handle it * Sync with main * Sync with main * add doc around dashboard telemetry (#2846) * wip * wip * update * add image * Revision based on suggestions, add to TOC * Apply suggestions from code review * Clarify command/resource type names not sent --------- Co-authored-by: Adam Ratzman <[email protected]> Co-authored-by: David Pine <[email protected]> * Clean up (#3378) * Clean up * Fix icon * Add content for user-assigned managed ids (#3377) * Initial draft of new user-assigned managed ids. * Edit pass and TOC reposition. * Fix issue with backticks * Minor updates * Correct URL to JSON config schema. (#3381) * Fix build warnings/errors (#3382) * Remove publisher APIs (#3389) * Fixes #3373, removal of publisher APIs. * Minor tweaks * Edit pass * Sort table * Apply suggestions from code review Co-authored-by: David Fowler <[email protected]> * Address feedback --------- Co-authored-by: David Fowler <[email protected]> * Add ACR hosting integration content. (#3383) * Add ACR hosting integration content. Fixes #3222 * Fix code ref * Address feedback * Add TOC preview * Adjust code bit * Fixes #3386, adds breaking changes content for Azure SQL Server admin user-assigned managed id. (#3391) * Fixes #3224. Move Dapr to community toolkit (#3393) * Adds new eventing APIs. (#3392) * Fixes #3372. Adds new eventing APIs. * Minor edit for constistency * Fix output * Fix build * Try to fix highlighting * Architecture content (#3156) * WIP * Early draft * Getting closer, still needs more refinement * Minor tweak * More updates * Added app topology examples * Add note about eventual consistency * Edit pass * Fix link * Apply suggestions from code review Co-authored-by: Andy (Steve) De George <[email protected]> * Address feedback * Address feedback * Fix up TOC * TOC adjustment * Clean up image * Add upgrade content to welcome * Adjust TOC * TOC tweak --------- Co-authored-by: Andy (Steve) De George <[email protected]> * Initial draft of Azure AI Inference content (#3399) * Initial draft of Azure AI Inference content * Tweak text * Tweak include text * Remove framework * Added logging and tracing sections. * Correct MD warning * Remove icon usage for alt-text * Trying something slightly different * Fix text * Apply suggestions from code review Co-authored-by: Andy (Steve) De George <[email protected]> * Update docs/azureai/azureai-inference-integration.md --------- Co-authored-by: Andy (Steve) De George <[email protected]> * Adds Azure App Configuration integration (#3500) * Fixes #3395, adds Azure App Configuration. * Fix include link * Apply suggestions from code review Co-authored-by: James Newton-King <[email protected]> * Apply suggestions from code review Co-authored-by: James Newton-King <[email protected]> * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Update dotnet-aspire-9.3.md * Add dashboard screenshots * Apply suggestions from code review Co-authored-by: Alex Crome <[email protected]> * A bit of clean up * Fix more links * Apply suggestions from code review Co-authored-by: Maddy Montaquila (Leger) <[email protected]> * Aspire 9.3 What's New Feedback Tweaking some wording to make the doc more correct. * Fun times... (#3505) * Add GHCP dashboard documentation * Update * Update what's new * Fixes --------- Co-authored-by: James Newton-King <[email protected]> * alerts --------- Co-authored-by: Andy (Steve) De George <[email protected]> Co-authored-by: Adam Ratzman <[email protected]> Co-authored-by: Adam Ratzman <[email protected]> Co-authored-by: David Fowler <[email protected]> Co-authored-by: James Newton-King <[email protected]> Co-authored-by: Alex Crome <[email protected]> Co-authored-by: Maddy Montaquila (Leger) <[email protected]> Co-authored-by: Eric Erhardt <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #2768
Internal previews