Skip to content

feat(react-router): Re-export functions from @sentry/react #16465

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Jun 4, 2025

Conversation

s1gr1d
Copy link
Member

@s1gr1d s1gr1d commented Jun 3, 2025

Re-exports functionality from the @sentry/react. However, not everything is exported as the react router exports would be confusing.

closes #16459

@s1gr1d s1gr1d requested review from chargome and RulaKhaled June 3, 2025 12:17
Copy link
Member

@chargome chargome left a comment

Choose a reason for hiding this comment

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

Thanks for adding this @s1gr1d 🚀

Comment on lines +12 to +13
ErrorBoundary,
withErrorBoundary,
Copy link
Member

Choose a reason for hiding this comment

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

nit: RR uses it's own error boundaries, not sure if we ever need these tbh 🤷

Copy link
Member Author

Choose a reason for hiding this comment

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

With the Sentry.ErrorBoundary you can configure some options around error handling like showing a report dialog. So I think it's worth exporting this :)

Copy link
Member

Choose a reason for hiding this comment

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

Go ahead then 👍

@s1gr1d s1gr1d merged commit ac22be2 into develop Jun 4, 2025
31 checks passed
@s1gr1d s1gr1d deleted the sig/react-router-react-exports branch June 4, 2025 08:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

react-router v7 SDK doesn't re-export from @sentry/react and is missing withProfiler
3 participants