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: Update build patch for react-native-background-upload #1232

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

zatteo
Copy link
Contributor

@zatteo zatteo commented Jul 18, 2024

react-native-background-upload is not maintained. We already had a patch for react-native 0.72, but we are getting an error when uploading a file when targeting Android 34 : "One of RECEIVER_EXPORTED or RECEIVER_NOT_EXPORTED should be specified when a receiver isn't being registered exclusively for system broadcasts".

Fix is done in android-upload-service
gotev/android-upload-service#642

New patch is taken from
Vydia/react-native-background-upload#347

### ✨ Features

*

### 🐛 Bug Fixes

* Crash when uploading a file on Android 14

### 🔧 Tech

*

Checklist

Before merging this PR, the following things must have been done if relevant:

  • Tested on iOS
  • Tested on Android
  • Test coverage
  • README and documentation

react-native-background-upload is not maintained. We already had a
patch for react-native 0.72, but we are getting an error when
uploading a file when targeting Android 34 : "One of
RECEIVER_EXPORTED or RECEIVER_NOT_EXPORTED should be specified when a
receiver isn't being registered exclusively for system broadcasts".

Fix is done in android-upload-service
gotev/android-upload-service#642

New patch is taken from
Vydia/react-native-background-upload#347
@zatteo zatteo merged commit e097ed5 into master Jul 18, 2024
1 check passed
@zatteo zatteo deleted the fix/android-34-upload branch July 18, 2024 14:16
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