-
Notifications
You must be signed in to change notification settings - Fork 902
Need for 0.5.0 version patch - use 4.22.1 version of facebook-android-sdk #339
Conversation
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks! If you are contributing on behalf of someone else (eg your employer): the individual CLA is not sufficient - use https://developers.facebook.com/opensource/cla?type=company instead. Contact cla@fb.com if you have any questions. |
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
@marysieek updated the pull request - view changes |
It is a good practice to use 4.22.1 version explicitly, to avoid regression in the future |
Same for 0.4.0 |
I've been battling this issue since Wednesday. Getting this PR in soon would be much appreciated. |
Also update your $root\android\app\build.gradle with explicit version if you have added fbsdk as dependency |
For the sake of completeness, here are the changes I made to get past this issue until the fix is in android\app\build.gradle
node_modules\react-native-fbsdk\android\build.gradle
|
this might make it a little easier to make the change repeatedly (after a new install or on other team members' machines):
(btw: @dev6rodn I think you meant this file |
Yes, that's correct, @jhilden. Thanks for catching that one. I just updated my previous comment. |
Can we get this merged asap? |
Please merge this PR as soon as possible.Our app is not working on production. |
ping @dzhuowen |
@vik228 in the meantime you could fork the latest working branch and apply this change to your forked version, then replace the dependency and voila! Actually I did that here, and in my project yarn add react-native-fbsdk@https://github.com/tugorez/react-native-fbsdk and everything is working fine. Hope it helps |
I can confirm that on version |
Instead of modifying the node_modules\react-native-fbsdk\android\build.gradle file, you can get the same outcome by modifying your root build.gradle. Again, for the sake on completeness, here's everything in one place. This is far better because you're free to remove and reinstall your node_modules\ if necessary. android\app\build.gradle
android\build.gradle
credit to the fine folks who commented here. |
I found it, I was trying to compile my app which is using facebook sdk. I was made that like augst 2016. When I try to open it today i got same error. I had that line in my gradle " compile 'com.facebook.android:facebook-android-sdk:4.+' " and I went https://developers.facebook.com/docs/android/change-log-4x this page and i found the sdk version while i was running this app succesfully and it was 4.14.1 then I changed that line to " compile 'com.facebook.android:facebook-android-sdk:4.14.1' " and it worked. |
Hi, android {
} dependencies { Thanks a lot |
Is this getting merged? |
@dzhuowen seems to be the one who merges pull requests. could we get this merged? react-native-fbsdk is currently unusable for anyone not able to upgrade their react-native version I believe |
Instead of setting the fbsdk to specific version, you can also update the android support library to 25.0.0 in android\app\build.gradle. This shouldn't change any current behavior of the app. dependencies { |
@adriancuco have you followed all these instructions? Your error indicates that you're missing an argument... that argument is described in that section. |
@rodnolan. Ahhh, understood. Let me check it out. Thx |
@adriancuco not related 😉 update your comment so that other readers will not get confused |
If you only up your own compileSdk and build tools, other libraries will complain, but this will help fix those, without changes inside node_modules: ext { subprojects { subproject -> |
I have my app running smoothly now with the latest versions of all my dependencies (react, react-native, react-native-fbsdk, etc) and I'm using version 23 of the sdk and build tools. I went through the upgrade process described here: http://facebook.github.io/react-native/docs/upgrading.html. It required some work but I'm glad I did it. I still have to lock down the Android Facebook SDK at version 4.22.1 but that's a minor adjustment, given the version mess I was creating by downgrading my dependencies. Here's my latest configuration. Your mileage may vary but this is what's working for me right now. package.json
android\build.gradle
android\app\build.gradle
|
@dev6rodn Is it still working fine for you? Tried exactly the same configuration with my build and got an error. I wonder this branch is not merged yet. |
@igorarkhipenko my app is still compiling without error. You may have to clean up a few things in order to get over the hump. Give this a try, if you haven't already.
|
@rodnolan unfortunately, didn't work for me neither. Anyway, thank you for your fast response, sir. Seems like, I've got a worm somewhere in my code. |
It was a lot of work but it was helpful to gradually rebuild my app from scratch. I started with a blank app and copied my old code into the new project piece by piece, chunk by chunk. I was very methodical in verifying every major step I took but I eventually found the finish line. Good luck. ;-) |
Is this going to be merged or not? I do not want to have to manually fix the Facebook SDK to version 4.22.1. |
Is there any plan to merge this back to master branch? My app is completely broken :( |
cc @dzhuowen |
Sorry for the inconvenience. We'll do a 0.6.1 release to fix sdk version. |
Did you mean 0.5.1? |
current version is 0.6.0 and he will push a bugfix out, that makes automatic versioning to 0.6.1.. |
But that means I need to upgrade my version of react-native-fbsdk to 0.6.0 just for this... |
Also I believe 0.6.0 requires RN 0.44 or later so it might require a react native upgrade as well. |
@cdimitroulas @farazs Thanks for info, |
👍 |
Why is 0.6.0 using build tools 25? As said by @marysieek up here it is not a good idea to up them. I guess I'll have to wait for 0.5.1 and stick to that. |
Still having the issue on 0.6.1. |
We upgraded to version |
React-native 0.44.0 and react-native-fbsdk -0.5.1 also works. |
I am still having this issue but I can see that version I can get my project to build using the workaround from this StackOverflow answer which proves that this is related to the react-native-fbsdk. Is it possible that this is a caching issue? I have tried a variety of different commands to try to get around this but to no avail. Would appreciate some help 😅 |
After much debugging, I updated all these: Now I am able to successfully compile with the latest version of FB SDK. |
I came from https://stackoverflow.com/questions/47671206/getting-error-when-run-react-native-run-android Somehow I got an error in android\build.gradle
|
Hi,
Facebook Android SDK was updated yesterday (facebook/facebook-android-sdk@d47ac93) and due to this fact, we cannot longer use
facebook-android-sdk 4.+
version in android/build.gradle for0.5.0
tag, since it results in regression.The issue is as follows:
Please release 0.5.1 version with fix applied. 😊