Skip to content

Conversation

@guardrex
Copy link
Collaborator

@guardrex guardrex commented Nov 10, 2025

Fixes #36315

Just need one review on this one.

Updates:

  • The versioning is a bit hairy 👹, so I'm going to have a little bit of duplication (a couple of sentences) to avoid version flipping.
  • Adding the fingerprinted asset links for the Blazor script.
  • Ordering change: Moving the bit on the RequiresAspNetWebAssets MSBuild prop UP because it logically follows discussing the server-side Blazor script location.

Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/project-structure.md aspnetcore/blazor/project-structure

@guardrex guardrex self-assigned this Nov 10, 2025
@guardrex guardrex requested a review from Copilot November 10, 2025 16:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reorganizes and updates the "Location of the Blazor script" section in the Blazor project structure documentation to improve clarity and version-specific information. The changes restructure content across multiple .NET versions to provide clearer guidance on where Blazor scripts are located.

Key Changes

  • Restructured the Blazor script location section with better version-specific organization using moniker ranges
  • Added information about WebAssembly script fingerprinting for .NET 10.0
  • Moved the RequiresAspNetWebAssets property guidance to a more logical position

@guardrex guardrex changed the title Improve the location of the Blazor script section [10.0] Improve the location of the Blazor script section Nov 10, 2025
@guardrex guardrex changed the title [10.0] Improve the location of the Blazor script section Improve the location of the Blazor script section Nov 10, 2025
@guardrex guardrex changed the title Improve the location of the Blazor script section [10.0] Improve the location of the Blazor script section Nov 10, 2025
Copy link
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

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

Looks great with the latest corrections. Approved.

@guardrex guardrex merged commit 25c532c into main Nov 10, 2025
4 checks passed
@guardrex guardrex deleted the guardrex/blazor-script-location branch November 10, 2025 20:37
@guardrex guardrex removed the request for review from tdykstra November 10, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[10.0] Provide script location examples with fingerprinting

3 participants