Skip to content
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

fix: expo users reported app crash on didFailToRegisterForRemoteNotificationsWithError #244

Merged
merged 2 commits into from Mar 5, 2024

Conversation

ami-aman
Copy link
Collaborator

@ami-aman ami-aman commented Mar 5, 2024

part of: https://linear.app/customerio/issue/MBL-141/expo-50-with-version-100-beta15-crash-startup-app-on-ios

This PR is using iOS SDK's version 2.12.3 with the fix for this crash. Refer this iOS PR for more detail.

@ami-aman ami-aman requested a review from a team March 5, 2024 15:08
@ami-aman
Copy link
Collaborator Author

ami-aman commented Mar 5, 2024

Added Do not merge label as deployment of the cocoa pods is in progress.

@ami-aman
Copy link
Collaborator Author

ami-aman commented Mar 5, 2024

Removed Do not merge label and tested the fix on sample app.

@ami-aman ami-aman merged commit fb9a464 into main Mar 5, 2024
3 of 5 checks passed
@ami-aman ami-aman deleted the aman/crash-fix-for-expo-user branch March 5, 2024 17:01
github-actions bot pushed a commit that referenced this pull request Mar 5, 2024
## [3.5.2](3.5.1...3.5.2) (2024-03-05)

### Bug Fixes

* expo users reported app crash on didFailToRegisterForRemoteNotificationsWithError ([#244](#244)) ([fb9a464](fb9a464))
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.

None yet

2 participants