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

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.


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