Skip to content

[docs] Update WithTerminal docs to reflect Terminal-view default for live resources - #1447

Merged
David Pine (IEvangelist) merged 1 commit into
release/13.5from
docs/pr-18867-terminal-default-view-c188035088789572
Aug 7, 2026
Merged

[docs] Update WithTerminal docs to reflect Terminal-view default for live resources#1447
David Pine (IEvangelist) merged 1 commit into
release/13.5from
docs/pr-18867-terminal-default-view-c188035088789572

Conversation

@aspire-repo-bot

Copy link
Copy Markdown
Contributor

Documents changes from microsoft/aspire#18867

@IEvangelist

Targeting release/13.5 based on the source PR milestone 13.5.

Why

PR microsoft/aspire#18867 changes the dashboard's Console logs page so that terminal-enabled resources (via WithTerminal()) default to the Terminal view when the resource is live (state Running), instead of always defaulting to Console logs. Non-live states (Waiting/Starting/exited) still default to Console logs. The existing docs page (app-host/with-terminal.mdx) described the old always-Console-logs-first behavior and needed correcting.

What changed

  • Updated the "View terminals in the dashboard" section of src/frontend/src/content/docs/app-host/with-terminal.mdx to describe the new state-dependent default view logic (Terminal when Running; Console logs when Waiting/Starting/Exited/Finished/FailedToStart), and clarified that a later state transition does not auto-switch the view once navigated.

Files modified

  • src/frontend/src/content/docs/app-host/with-terminal.mdx

No new pages were created; this is an update to an existing page.

Generated by PR Documentation Check · auto · 29.3 AIC · ⌖ 9.54 AIC · ⊞ 17.7K ·

Documents microsoft/aspire#18867

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@aspire-repo-bot aspire-repo-bot Bot added the docs-from-code Copilot initiated issue from dotnet/aspire repo label Aug 7, 2026
@IEvangelist
David Pine (IEvangelist) marked this pull request as ready for review August 7, 2026 07:35
Copilot AI lite review requested due to automatic review settings August 7, 2026 07:35

@IEvangelist David Pine (IEvangelist) 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.

Validated against microsoft/aspire#18867. The state-dependent default, selection-time evaluation, and preserved live views are documented accurately.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Aspire AppHost WithTerminal() documentation to match the Aspire dashboard behavior change where terminal-enabled resources default to the Terminal view when the resource is live (Running), and default to Console logs for non-live states.

Changes:

  • Documented state-dependent default view selection on the dashboard Console Logs page for terminal-enabled resources.
  • Clarified that view selection is evaluated when navigating to a resource and does not auto-switch on later state transitions.
  • Retained guidance on manually switching views via the toolbar options (⋯) menu.

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

@aspire-repo-bot

Copy link
Copy Markdown
Contributor Author

Frontend HTML artifact ready

The latest frontend build uploaded the frontend-dist artifact for PR #1447. Use the VS Code button below to open this PR with GitHub Artifacts Explorer and browse the built HTML locally.

VS Code: Open PR #1447 artifacts

This comment updates automatically when a new frontend build artifact is uploaded.

@IEvangelist
David Pine (IEvangelist) merged commit c52c4e3 into release/13.5 Aug 7, 2026
17 of 20 checks passed
@IEvangelist
David Pine (IEvangelist) deleted the docs/pr-18867-terminal-default-view-c188035088789572 branch August 7, 2026 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-from-code Copilot initiated issue from dotnet/aspire repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants