Skip to content

fix: remove deprecated push notification ios export#516

Merged
Brentlok merged 1 commit intomainfrom
remove-deprecated
Apr 27, 2026
Merged

fix: remove deprecated push notification ios export#516
Brentlok merged 1 commit intomainfrom
remove-deprecated

Conversation

@Brentlok
Copy link
Copy Markdown
Contributor

@Brentlok Brentlok commented Apr 27, 2026

fixes #509

Summary by CodeRabbit

  • Breaking Changes
    • Removed PushNotificationIOS export from the components module.

@Brentlok Brentlok merged commit 0eaf017 into main Apr 27, 2026
1 of 2 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 27, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 16dc6c9d-0c30-4a7e-8c87-8b0bc80ce209

📥 Commits

Reviewing files that changed from the base of the PR and between c62f834 and 06ed158.

📒 Files selected for processing (1)
  • packages/uniwind/src/components/index.ts

📝 Walkthrough

Walkthrough

The PushNotificationIOS getter is removed from the exported proxy object in packages/uniwind/src/components/index.ts. This deprecated export from react-native was causing release build crashes on iOS due to eager evaluation in production builds.

Changes

Cohort / File(s) Summary
Removed Deprecated Export
packages/uniwind/src/components/index.ts
Removed the PushNotificationIOS getter that re-exported deprecated functionality from react-native, which was triggering crashes during release builds on iOS.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A getter that crashed in the light,
Has been removed to set things right.
No more eager evaluation's plight,
Release builds now shine so bright! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove-deprecated

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Brentlok Brentlok deleted the remove-deprecated branch April 27, 2026 06:04
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.

Release build crash: PushNotificationIOS re-export from react-native core

1 participant