镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

fix for https://github.com/wix/react-native-notifications/issues/1071… - #1075

Open
nhoffmann-fts wants to merge 1 commit into
wix:masterfrom
nhoffmann-fts:fix_react-native_82
Open

nhoffmann-fts wants to merge 1 commit into
wix:masterfrom
nhoffmann-fts:fix_react-native_82

Conversation

@nhoffmann-fts

Copy link
Copy Markdown

… . Applied patch to this branch as described in issue.

@sahityaPasnoor

Copy link
Copy Markdown

Confirmed this fixes the exact crash from #1071 on our end — react-native 0.85.2, newArchEnabled=true (Bridgeless). Without this, FcmToken.sendTokenToJS()'s unconditional getReactNativeHost().getReactInstanceManager() call throws UnsupportedOperationException immediately on token refresh, before ever reaching the getReactHost() fallback. Applied this diff as a local yarn patch and confirmed the token now delivers to JS without crashing. Would be great to see this merged — it's blocking Android push notifications for any app on the New Architecture.

@stale

stale Bot commented Oct 31, 2026

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the 🏚 stale label Oct 31, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants