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

Unexpected error on render <CachedImage /> #129

Open
joaom182 opened this issue Jul 19, 2018 · 8 comments
Open

Unexpected error on render <CachedImage /> #129

joaom182 opened this issue Jul 19, 2018 · 8 comments

Comments

@joaom182
Copy link

joaom182 commented Jul 19, 2018

Error in callNativeModules();
Malformed calls from JS: fields sizes are different.

screen shot 2018-07-19 at 03 14 04

@joaom182
Copy link
Author

joaom182 commented Jul 19, 2018

This issue is caused by the latest version is using react-native-fetch-blob instead of using rn-fetch-blob.

@fungilation makes a fork that's correct the dependency, so i sugest to use this fork.

  • npm uninstall react-native-cached-image --save
  • Check if the folder react-native-cached-image still on node_modules and remove completely.
  • npm uninstall react-native-fetch-blob --save
  • react-native unlink react-native-fetch-blob
  • npm install rn-fetch-blob --save
  • react-native link rn-fetch-blob
  • npm install fungilation/react-native-cached-image --save
  • ❤️

@JensDebergh
Copy link

If react-native-fetch-blob changed their name to rn-fetch-blob should we just merge @fungilation's fork and merge it into master?

The library react-native-cached-image is completely broken right now on latest versions of react-native.

react-native-fetch-blob is officially discontinued in favor of rn-fetch-blob

@ARMATAV
Copy link

ARMATAV commented Nov 27, 2018

@JensDebergh yes please - this is causing all sorts of issues, including breaking release builds for Android

@cyqui
Copy link

cyqui commented Feb 26, 2019

The project according to the contributors charts and the last commits seem rather dead,so unfortunately i have the feeling the PR is not going to be merged any soon.

https://github.com/kfiroo/react-native-cached-image/graphs/contributors

@kfiroo
Would be great if the README could mention to use fungilation/react-native-cached-image or any more popular fork you'll recommend. All the failed installs are consuming a lot of power energy and we are killing polar bears by giving outdated code without warnings.

@venux92
Copy link

venux92 commented Mar 16, 2019

@joaom182 Great, been trying to debug it all day you just made my day

@andho
Copy link

andho commented Apr 20, 2019

netinfo package is needed now.
fungilation@363dea9

@fungilation
Copy link
Collaborator

fungilation commented Apr 20, 2019

I know. See my last commit on my fork, tested to work with up to RN 0.59.5: https://github.com/fungilation/react-native-cached-image/commits/master

@fungilation
Copy link
Collaborator

On topic of maintenance and forking: #160

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

No branches or pull requests

7 participants