Skip to content

Conversation

@jamescrosswell
Copy link
Collaborator

DESCRIBE YOUR PR

Resolves getsentry/sentry-dotnet#4497

IS YOUR CHANGE URGENT?

  • None: Not urgent, can wait up to 1 week+

SLA

cc: @Flash0ver

PRE-MERGE CHECKLIST

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

@vercel
Copy link

vercel bot commented Nov 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
develop-docs Ready Ready Preview Comment Nov 6, 2025 9:39pm
sentry-docs Ready Ready Preview Comment Nov 6, 2025 9:39pm

@codeowner-assignment codeowner-assignment bot requested a review from a team November 3, 2025 03:01
cursor[bot]

This comment was marked as outdated.

@codecov
Copy link

codecov bot commented Nov 3, 2025

Bundle Report

Changes will decrease total bundle size by 12.49kB (-0.05%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-client-array-push* 10.16MB -6 bytes (-0.0%) ⬇️
sentry-docs-server-cjs 12.92MB -12.49kB (-0.1%) ⬇️

ℹ️ *Bundle size includes cached data from a previous commit

Affected Assets, Files, and Routes:

view changes for bundle: sentry-docs-client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
static/chunks/pages/_app-*.js -3 bytes 882.71kB -0.0%
static/chunks/8321-*.js -3 bytes 425.87kB -0.0%
static/9thbyVRFs9Z4DwCJTK3oV/_buildManifest.js (New) 684 bytes 684 bytes 100.0% 🚀
static/9thbyVRFs9Z4DwCJTK3oV/_ssgManifest.js (New) 77 bytes 77 bytes 100.0% 🚀
static/vgAVzbITWufmZ_YvgQqr2/_buildManifest.js (Deleted) -684 bytes 0 bytes -100.0% 🗑️
static/vgAVzbITWufmZ_YvgQqr2/_ssgManifest.js (Deleted) -77 bytes 0 bytes -100.0% 🗑️
view changes for bundle: sentry-docs-server-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
1729.js -3 bytes 1.74MB -0.0%
../instrumentation.js -3 bytes 1.07MB -0.0%
9523.js -3 bytes 1.04MB -0.0%
../app/[[...path]]/page.js.nft.json -4.16kB 872.68kB -0.47%
../app/platform-redirect/page.js.nft.json -4.16kB 872.6kB -0.47%
../app/sitemap.xml/route.js.nft.json -4.16kB 869.83kB -0.48%

{
source: '/platforms/dotnet/guides/uwp/:path*',
destination: '/platforms/dotnet/guides/maui/',
},
Copy link

Choose a reason for hiding this comment

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

Bug: Redirection Chain: UWP to Maui Needs Cleanup

The new redirect for UWP paths creates a redirect chain. There's an existing redirect in src/middleware.ts (line 925) that redirects /platforms/dotnet/uwp/ to /platforms/dotnet/guides/uwp/. When combined with this new redirect from /platforms/dotnet/guides/uwp/:path* to /platforms/dotnet/guides/maui/, users accessing /platforms/dotnet/uwp/ will be redirected twice: first by middleware to /platforms/dotnet/guides/uwp/, then by this redirect to /platforms/dotnet/guides/maui/. The middleware redirect should be removed to create a direct single redirect from /platforms/dotnet/uwp/ to /platforms/dotnet/guides/maui/.

Fix in Cursor Fix in Web

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.

Deprecate UWP for v6

3 participants