Skip to content

Upgrade Blazor Server AdvancedSearch sample to .NET 8#33

Open
devin-ai-integration[bot] wants to merge 1 commit into
masterfrom
devin/1777990995-net8-blazor-advancedsearch
Open

Upgrade Blazor Server AdvancedSearch sample to .NET 8#33
devin-ai-integration[bot] wants to merge 1 commit into
masterfrom
devin/1777990995-net8-blazor-advancedsearch

Conversation

@devin-ai-integration
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot commented May 5, 2026

Summary

Upgrades AspNetCore/Blazor/AdvancedSearch.BlazorServer/EqDemo.BlazorServer.AdvancedSearch.csproj from .NET 6 to .NET 8 as part of the coordinated multi-session migration. Only this single project is touched; no other .csproj, root files, or sibling projects were modified.

.csproj changes

Item Before After
<TargetFramework> net6.0 net8.0
Microsoft.EntityFrameworkCore.Sqlite 6.0.1 8.0.26
Microsoft.EntityFrameworkCore.SqlServer 6.0.1 8.0.26
Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore 6.0.1 8.0.26
Microsoft.Data.SqlClient 2.1.7 5.2.3
Microsoft.IdentityModel.JsonWebTokens 6.34.0 7.7.1
System.IdentityModel.Tokens.Jwt 6.34.0 7.7.1
System.Data.SqlClient 4.8.6 removed (replaced by Microsoft.Data.SqlClient / .NET 8 BCL)
System.Net.Http 4.3.4 removed (provided by .NET 8 BCL)
System.Text.RegularExpressions 4.3.1 removed (provided by .NET 8 BCL)
System.Drawing.Common 4.7.2 removed (provided by .NET 8 BCL)

System.Drawing.Common was on the removal list per the task spec; on .NET 8 the type ships in the runtime/shared framework so the explicit reference is no longer needed.

Code changes

None required. _Imports.razor references @using System.Net.Http, but the namespace is part of the BCL so removing the package reference does not affect the build. No other source files in AspNetCore/Blazor/AdvancedSearch.BlazorServer/ referenced any of the removed packages.

Build verification

Ran the following on the VM (using .NET SDK 8.0.420 installed via dotnet-install.sh):

dotnet restore AspNetCore/Blazor/AdvancedSearch.BlazorServer/EqDemo.BlazorServer.AdvancedSearch.csproj
dotnet build   AspNetCore/Blazor/AdvancedSearch.BlazorServer/EqDemo.BlazorServer.AdvancedSearch.csproj -c Release

Result:

Build succeeded.
    0 Warning(s)
    0 Error(s)

CI

CI status will be polled with git(action="pr_checks", wait_mode="all") after the PR is opened and reported in the final session message.

Review & Testing Checklist for Human

  • Confirm dotnet build -c Release on EqDemo.BlazorServer.AdvancedSearch.csproj succeeds in your environment with the .NET 8 SDK.
  • Smoke-test the Blazor Server app locally: run it, navigate to the Advanced Search page, build a query against the bundled NWind sample DB, and confirm results render and Excel/PDF export still work (the EasyData/Korzh packages are unchanged but their transitive dependencies pulled by EF Core 8 differ).
  • Verify SQL Server connectivity still works if you use the SqlServer provider (Microsoft.Data.SqlClient bumped from 2.1.7 → 5.2.3 — connection strings using legacy encryption defaults may need Encrypt=False or a server cert update; see Microsoft.Data.SqlClient 4.0+ breaking changes).

Notes

  • Scope strictly limited to the single .csproj listed above. No global.json, Directory.Build.props, .sln, or other projects were touched.
  • Package versions chosen as the latest stable 8.0.x / 5.2.x / 7.x available on NuGet.org at the time of the upgrade.

Link to Devin session: https://app.devin.ai/sessions/f94354c6acc24c58906392b70ccdc031
Requested by: @tobydrinkall


Devin Review

Status Commit
⚪ Not started

Run Devin Review

💡 Connect your GitHub account to enable automatic code reviews.

Open in Devin Review (Staging)
Open in Devin Review

Co-Authored-By: Toby Drinkall <toby.drinkall@cognition.ai>
@devin-ai-integration
Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Copy Markdown
Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

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.

1 participant