Skip to content

Update react native unhandled js exception handling #2398

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 7, 2025

Conversation

gingerbenw
Copy link
Member

@gingerbenw gingerbenw commented Apr 3, 2025

Goal

Update react native exception handling to discard double reported errors caused by unhandled js errors

Changeset

Update expected string in error discard code when using new architecture

Testing

Existing ci tests

Copy link

github-actions bot commented Apr 3, 2025

@bugsnag/browser bundle size diff

Minified Minfied + Gzipped
Before 46.96 kB 14.02 kB
After 46.96 kB 14.02 kB
± No change No change

code coverage diff

<temporarily disabled>

Generated by 🚫 dangerJS against 0683b67

@gingerbenw gingerbenw force-pushed the gingerbenw/react-native-78-ios-fix branch 2 times, most recently from 01305a6 to 04d20c2 Compare April 4, 2025 11:48
@gingerbenw gingerbenw force-pushed the gingerbenw/react-native-78-ios-fix branch 2 times, most recently from f278fc0 to 3c1bc2d Compare April 4, 2025 14:44
@gingerbenw gingerbenw changed the title refactor: ♻️ update expected unhandled exception string on new… Update react native unhandled js exception handling Apr 7, 2025
@gingerbenw gingerbenw marked this pull request as ready for review April 7, 2025 11:31
@gingerbenw gingerbenw merged commit 72e814f into next Apr 7, 2025
24 checks passed
@gingerbenw gingerbenw mentioned this pull request Apr 9, 2025
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.

2 participants