Refine integrations docs#1055
Open
davidfowl wants to merge 2 commits into
Open
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refines the integrations documentation to focus on AppHost resource modeling and references (rather than client integrations / Service Defaults), and removes Aspire.Hosting.Testing from the integrations catalog + related metadata used by the gallery/autocomplete.
Changes:
- Rewrites the integrations overview docs to describe integrations as AppHost packages that model resources and pass connection details via references/environment variables.
- Removes the “Built-in superpowers”/Service Defaults feature section from the integrations landing page to avoid overly broad capability claims.
- Excludes
Aspire.Hosting.Testingfrom integrations data sources and updates gallery no-results localized strings to no longer suggest searching for “Testing”.
Show a summary per file
| File | Description |
|---|---|
| src/frontend/src/data/integration-docs.json | Removes the Aspire.Hosting.Testing → /testing/overview/ docs mapping so it’s no longer treated as an integration doc target. |
| src/frontend/src/data/aspire-integrations.json | Removes Aspire.Hosting.Testing from the generated integrations catalog data. |
| src/frontend/src/data/aspire-integration-names.json | Removes Aspire.Hosting.Testing from integration name/autocomplete data. |
| src/frontend/src/content/i18n/zh-CN.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/uk.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/tr.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/ru.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/pt-BR.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/ko.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/ja.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/it.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/id.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/hi.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/fr.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/es.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/en.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/de.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/i18n/da.json | Updates gallery “no results” suggestion text to use “Redis” instead of “Testing”. |
| src/frontend/src/content/docs/integrations/overview.mdx | Reframes integrations around AppHost resource modeling + references and updates the environment variable explanation/examples. |
| src/frontend/src/content/docs/integrations/index.mdx | Removes the Service Defaults/“superpowers” showcase section and softens capability claims in the hero copy. |
| src/frontend/scripts/update-integrations.ts | Excludes Aspire.Hosting.Testing from the integrations data generation script going forward. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 21/21 changed files
- Comments generated: 2
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
|
Thanks for the summary. This matches the intended scope of the PR, so no additional changes are needed for this overview comment. |
Member
|
@davidfowl should this be targetting release/13.4 branch? |
Contributor
Author
|
Main |
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
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.
The integrations pages were still framing integrations around client integrations and Service Defaults, which made the AppHost-focused story less clear and too .NET-specific. This updates the docs to define integrations as AppHost packages for modeling resources and passing connection details.
Summary
Aspire.Hosting.Testingfrom the integrations catalog and related docs/autocomplete data.Validation
/integrations/,/integrations/overview/, and/integrations/gallery/locally with Playwright.