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

Replace react-native-fetch-blob with rn-fetch-blob #150

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

shockdesign
Copy link

I know this was done in #144 however it didn't look complete. This one should pass all tests, etc.

@shockdesign
Copy link
Author

@kfiroo Hey! I'm just wondering if there is any movement on any of the PRs here? Let me know if I can help?

@Mahdeep
Copy link

Mahdeep commented Jan 17, 2019

Please Merge this Commit

@fawaz-ahmed
Copy link

fawaz-ahmed commented Mar 6, 2019

Till this commit is merged, we can use a workaround.

yarn add -D babel-plugin-module-resolver

and in your .babelrc add the following :

"plugins": [
  ["module-resolver", {
    "alias": {
      "react-native-fetch-blob": "rn-fetch-blob"
    }
  }]
]

Hope this helps.

@nicomontanari
Copy link

@kfiroo please merge this pr

@ShadowManu
Copy link

@fungilation I know you just received access to the repo by #160, but I do believe this is high on the priority list of users (including me :D)

@fungilation
Copy link
Collaborator

fungilation commented May 7, 2019

My #124 did the equivalent and is on my fork at https://github.com/fungilation/react-native-cached-image. I don't have time this week, maybe next week on merging my fork back into here so until then, my fork works and tested up to RN 0.59.5

@fungilation
Copy link
Collaborator

Note to self: not closing this PR as I can use the README update here

@stvmachine
Copy link

Till this commit is merged, we can use a workaround.

yarn add -D babel-plugin-module-resolver

and in your .babelrc add the following :

"plugins": [
  ["module-resolver", {
    "alias": {
      "react-native-fetch-blob": "rn-fetch-blob"
    }
  }]
]

Hope this helps.

I tried this but don't work for me :(.

@luciapuccini
Copy link

@kfiroo Hi, please merge this PR.

package.json Outdated Show resolved Hide resolved
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.

9 participants