-
Notifications
You must be signed in to change notification settings - Fork 467
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
Random Crashes #121
Comments
…hich crash app in production updated some dependencies, including switching unmaintained react-native-fetch-blob to rn-fetch-blob - fixes kfiroo#122, kfiroo#121
Yeah, all this stuff is fixed there, I'm using own fork which is already merged to rn-fetch-blob, but can't change react-native-fetch-blob to rn-fetch-blob cause cached-image using it so I must still use my own fork:
if interested. This way you will have native code with fixes, even cached-image dependency is for different version (and repo). But better will be to use ready PRs repo commit:
which I will test now and probably use. There is also yarn way:
but above PR also uses Promises to reject instead throwing exception in fsUtils (react-native-cached-image). |
Note that to use my fork (fungilation/), change native installation in your own app to I'm using in my app WonderSwipe without issue. |
I have presented multiple crashes in my application for many users. Currently the app has a large number of users, which makes these sporadic crashes more visible.
Following up with the help of AppSee, I realized that a large percentage of crashes are on the screens where images are loaded with this library. This affects less than 2% of the users, but they are enough to be taken into account.
I was also able to replicate them in the simulator in release mode and I was able to determine what is caused by
react-native-fetch-blob
.Following this problem, I realized that the only place where
react-native-fetch-blob
is as a dependency is in this library, also reviewing this module I realized that it seems that they are not maintaining it. It seems that now must be usedrn-fetch-blob
https://github.com/wkh237/react-native-fetch-blob#rnfb-maintainer-gone-missing-help-wanted
I would greatly appreciate that you follow this thread, since this bookstore has been very helpful to me.
The text was updated successfully, but these errors were encountered: