Remove unsupported blazorserver sample #65320
Merged
ilonatommy merged 2 commits intodotnet:mainfrom Feb 5, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the legacy BlazorServerApp sample under src/Components/Samples, consistent with the move to consolidated Blazor templates in newer .NET versions.
Changes:
- Deletes the
BlazorServerAppsample project (.csproj) and its app code (startup, pages, layouts, components). - Removes associated configuration files (appsettings, launchSettings) and static web assets (CSS, bootstrap icons, fonts, favicon).
Reviewed changes
Copilot reviewed 23 out of 28 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/Components/Samples/BlazorServerApp/wwwroot/favicon.ico | Removed along with the obsolete sample’s static assets. |
| src/Components/Samples/BlazorServerApp/wwwroot/css/site.css | Removed sample styling. |
| src/Components/Samples/BlazorServerApp/wwwroot/css/bootstrap-icons/fonts/bootstrap-icons.woff2 | Removed bootstrap icon font asset. |
| src/Components/Samples/BlazorServerApp/wwwroot/css/bootstrap-icons/fonts/bootstrap-icons.woff | Removed bootstrap icon font asset. |
| src/Components/Samples/BlazorServerApp/wwwroot/css/bootstrap-icons/bootstrap-icons.scss | Removed bootstrap icons source asset. |
| src/Components/Samples/BlazorServerApp/wwwroot/css/bootstrap-icons/bootstrap-icons.min.css | Removed bootstrap icons CSS asset. |
| src/Components/Samples/BlazorServerApp/wwwroot/css/bootstrap-icons/bootstrap-icons.json | Removed bootstrap icons metadata asset. |
| src/Components/Samples/BlazorServerApp/appsettings.json | Removed sample configuration. |
| src/Components/Samples/BlazorServerApp/appsettings.Development.json | Removed sample development configuration. |
| src/Components/Samples/BlazorServerApp/_Imports.razor | Removed sample Razor imports. |
| src/Components/Samples/BlazorServerApp/Startup.cs | Removed sample Startup configuration. |
| src/Components/Samples/BlazorServerApp/Shared/NavMenu.razor | Removed sample navigation component. |
| src/Components/Samples/BlazorServerApp/Shared/MainLayout.razor | Removed sample layout component. |
| src/Components/Samples/BlazorServerApp/Properties/launchSettings.json | Removed sample launch settings. |
| src/Components/Samples/BlazorServerApp/Program.cs | Removed sample host bootstrap. |
| src/Components/Samples/BlazorServerApp/Pages/_Layout.cshtml | Removed sample host layout page. |
| src/Components/Samples/BlazorServerApp/Pages/_Host.cshtml | Removed sample Blazor host page. |
| src/Components/Samples/BlazorServerApp/Pages/Index.razor | Removed sample page. |
| src/Components/Samples/BlazorServerApp/Pages/FetchData.razor | Removed sample page and data-fetching demo. |
| src/Components/Samples/BlazorServerApp/Pages/Error.razor | Removed sample error page. |
| src/Components/Samples/BlazorServerApp/Pages/Counter.razor | Removed sample counter page. |
| src/Components/Samples/BlazorServerApp/Data/WeatherForecastService.cs | Removed sample data service. |
| src/Components/Samples/BlazorServerApp/Data/WeatherForecast.cs | Removed sample data model. |
| src/Components/Samples/BlazorServerApp/BlazorServerApp.csproj | Removed the sample project definition. |
| src/Components/Samples/BlazorServerApp/App.razor | Removed sample router/app root component. |
Comments suppressed due to low confidence (2)
src/Components/Samples/BlazorServerApp/BlazorServerApp.csproj:1
- The repo still references the removed BlazorServerApp sample in developer tooling/docs (e.g., src/Components/.vscode/tasks.json has a BlazorServerApp-build task pointing at Samples/BlazorServerApp/BlazorServerApp.csproj; src/Components/.vscode/launch.json has a BlazorServerApp launch config; src/Components/AGENTS.md lists BlazorServerApp as an available sample). Please update/remove those references as part of the sample deletion so contributors don't get broken tasks/docs.
src/Components/Samples/BlazorServerApp/BlazorServerApp.csproj:1 - PR description mentions removing the legacy
dotnet new blazorwasmsample, but this change set deletes BlazorServerApp (Blazor Server template). Please align the description/title with what’s actually being removed (or include the intended Blazor WASM sample removal if that’s the goal).
Member
|
there are a couple more cases of |
This was referenced Feb 4, 2026
Open
pavelsavara
approved these changes
Feb 4, 2026
akoeplinger
approved these changes
Feb 4, 2026
blazorserever sample blazorserver sample
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.
Discussion initiated here: #65292 (comment)
.Net 8 consolidated the template and in #46996 we added the united blazor sample to the repository.
dotnet new blazorwasmworks for versions 7 and lower..Net 7 finished its support in 2024. I think this sample can be removed from the repository.