Skip to content

fix: [.NET 10] PageNavigationExtensions updated for public Page properties#5278

Open
nor0x wants to merge 3 commits into
getsentry:mainfrom
nor0x:fix-4578
Open

fix: [.NET 10] PageNavigationExtensions updated for public Page properties#5278
nor0x wants to merge 3 commits into
getsentry:mainfrom
nor0x:fix-4578

Conversation

@nor0x
Copy link
Copy Markdown

@nor0x nor0x commented Jun 2, 2026

starting with .NET 10 DestinationPage and PreviousPage in NavigatedFromEventArgs and NavigatedToEventArgs are no longer internal so no need for getting them via reflection in PageNavigationExtensions.

PR in dotnet/maui: dotnet/maui#28384
fixes: #4578

starting with .NET 10 DestinationPage and PreviousPage in NavigatedFromEventArgs and NavigatedToEventArgs are no longer internal so no need for getting them via reflection in PageNavigationExtensions.

see: getsentry#4578
and: dotnet/maui#28384
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8d5b5ee. Configure here.

Comment thread src/Sentry.Maui/Internal/PageNavigationExtensions.cs Outdated
Comment thread src/Sentry.Maui/Internal/PageNavigationExtensions.cs Outdated
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.

Missing breadcrumbs for PreviousPage in .NET 10

1 participant