Skip to content
This repository was archived by the owner on Mar 16, 2019. It is now read-only.

Add okhttp3 to build.gradle #238

Open
wants to merge 4 commits into
base: 0.10.2
Choose a base branch
from

Conversation

petterh
Copy link
Contributor

@petterh petterh commented Jan 20, 2017

Thank you for making a pull request ! Just a gentle reminder :)

  1. If the PR is offering a feature please make the request to our "Feature Branch" 0.11.0
  2. Bug fix request to "Bug Fix Branch" 0.10.2
  3. Correct README.md can directly to master

gferreyra91 and others added 3 commits January 20, 2017 10:42
Fixed typing errors
Code wouldn't work with those lines in there. They were meant to be comments.
@petterh
Copy link
Contributor Author

petterh commented Jan 20, 2017

Just a note: I messed up my git branching a bit; the 'fix readme' and 'commented out header lines' commits are not mine. If this causes any problems, let me know and I'll redo the pull request.

@bramus
Copy link

bramus commented May 4, 2017

Can this PR be merged please? Using master from react-native fails with this error.

@@ -34,5 +34,6 @@ android {

dependencies {
compile 'com.facebook.react:react-native:+'
compile 'com.squareup.okhttp3:okhttp:3.4.1'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There's a React Native PR to upgrade okhttp to 3.6.0 (or possibly 3.7.0). If and when that one is merged in, the version should change here as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

@petterh They have now updated to 3.8.0

@lll000111
Copy link
Contributor

lll000111 commented Aug 14, 2017

I second the request. The referenced ticket has been closed, and I had to do the same thing when I started working on the Android sources. I still get quite a few other errors in Android Studio (https://gist.github.com/lll000111/423b858d707daf5eba5212ca5e1f6baa), which makes me wonder what else is needed to setup the Android development environment... no problems running a project, but the dev. environment is messed up.

@lll000111 lll000111 mentioned this pull request Aug 14, 2017
@pencilcheck
Copy link

pencilcheck commented Oct 19, 2017

Would like to see this merged as it could fix a lot of issues regarding missing imports.

@petterh petterh force-pushed the add-okhttp3-to-build.gradle branch from 7eb1ff9 to 71fd9f5 Compare October 19, 2017 08:54
@lll000111
Copy link
Contributor

FYI

react-native-fetch-blob has a new maintained location. Check out the README.

Please consider submitting your PR there, this repository is unmaintained.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants