Skip to content

Bugfix/ios fix create file base64 #600

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 8 commits into from
Apr 4, 2022

Conversation

nadav2051
Copy link

No description provided.

nadav2051 and others added 8 commits June 3, 2019 14:56
joltup#369

Resolution to the above issue, the RN code passes two instances of Content-Type, one containing capital letters and the other in lower case. 

The Android native code saves those options after applying .toLowerCase() on them.

The data in the lower case content-type overwrote the data in the Content-Type, and since the data on both the keys wasn't updated together at the fixed part of the code, it overwrote it with invalid data and caused the request to fail.
Syncing with main branch, support for 0.60
Intent.FLAG_ACTIVITY_NEW_TASK added
Solved an issue where the rootNavigationController is not necessarily the currently displayed controller, which caused an error of:
Warning: Attempt to present <SecondViewController: 0x7fb54b523240> on <ViewController: 0x7fb54b61e7f0> whose view is not in the window hierarchy!
Android to throw an exception when there's no suitable app that can open the file
# Conflicts:
#	android/src/main/java/com/RNFetchBlob/RNFetchBlob.java
Copy link

@nomadx nomadx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

working

@rpenfold rpenfold merged commit dfc6806 into joltup:master Apr 4, 2022
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.

5 participants