Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 17, 2025

Add page reload on circuit resume failure for remote pauses

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Simplified version of #64290 applying fixes from fce8418 without graceful pause handling

Description

In .NET 10, circuit resume fails after server restart without persistent storage. Current behavior shows reconnection UI requiring manual page reload, regressing from .NET 9's automatic reload.

Changes:

  • DefaultReconnectDisplay.ts: Renamed reloadButtonretryButton for consistency with button text
  • UserSpecifiedDisplay.ts: Track remote property and include it in resume-failed events
  • ReconnectModal.razor.js: Auto-reload on resume-failed when remote === true, indicating server-initiated pause

Flow:

Server restart → reconnect fails → resume attempted with remote: true → resume fails → resume-failed event fires with remote: true → page reloads automatically

Restores .NET 9 behavior for local development scenarios.

Fixes #64228

Original prompt

Create a simplified PR from #64290 that doesn't include the graceful changes and that only contains the fixes in fce8418 and where this change
} else if (event.detail.state === "rejected" || (event.detail.state === "resume-failed" && !!event.detail.graceful)) { checks for the .remote property (remote == true) instead


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Simplify PR by removing graceful changes Add page reload on circuit resume failure for remote pauses Nov 17, 2025
Copilot AI requested a review from javiercn November 17, 2025 18:27
Copilot finished work on behalf of javiercn November 17, 2025 18:27
@javiercn javiercn closed this Nov 18, 2025
@dotnet-policy-service dotnet-policy-service bot added this to the 11.0-preview1 milestone Nov 18, 2025
@snympi
Copy link

snympi commented Nov 19, 2025

@javiercn was this change committed?

@javiercn
Copy link
Member

@snympi No.

This was me playing with some stuff. The original PR is tracking the fix.

@javiercn javiercn deleted the copilot/fix-remote-property-check branch November 19, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Blazor Server app in .NET 10 fails to reconnect after server restart, unlike .NET 9

3 participants