Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
Bundle ReportChanges will increase total bundle size by 328.76kB (1.42%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-client-array-pushAssets Changed:
view changes for bundle: sentry-docs-server-cjsAssets Changed:
|
| { | ||
| source: '/platforms/dotnet/guides/uwp/:path*', | ||
| destination: '/platforms/dotnet/guides/maui/', | ||
| }, |
There was a problem hiding this comment.
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/.
3 participants
DESCRIBE YOUR PR
Resolves getsentry/sentry-dotnet#4497
IS YOUR CHANGE URGENT?
SLA
cc: @Flash0ver
PRE-MERGE CHECKLIST