feat: wrap initialize sdk function

[rnike]

From issue facebookarchive/react-native-fbsdk#840

I wrap the initialize function from native, the original pr is facebookarchive/react-native-fbsdk#841

[thebergamo]

I looks fine, but I believe it would be good to have some note in the README.md about it that you should do it before start using it.

[mikehardy]

Yes! I believe this would stop the crash I saw and worked around by adding an AppDelegate chunk. +1 on a README note as otherwise I would have no idea, maybe even a major version bump

[rnike]

README is added, any suggestions if it's not clear.

[rnike]

@mikehardy I see, that's why you've added the code in didFinishLaunchingWithOptions.

[thebergamo]

Besides comments, looks like there is some error in iOS build.
@rnike Anyway, thanks for this PR, I totally forgot about it while just upgrading version.

@mikehardy I agree with you. this could be a major bump.

[rnike]

The AutoInitEnabled default is true if not set on Android. If following GDPR-style we will need to also wrap FacebookSdk.setAutoInitEnabled for Android (reference).

In this pr I will be focused on sdk initialization.

[thebergamo]

I believe now everything looks good to me :)

[rnike]

just accidentally unassigned myself, don't mine it

[thebergamo]

Fixed :)

[thebergamo]

Let's just wait for @mikehardy feedback on it and I'm good to merge and release it.

[mikehardy]

I have no problems with the code (it's easy code thankfully!) but I made a suggestion to restructure the document with the idea it will nudge users to be platform-neutral and use the Javascript code. And I expanded the information about GDPR vs non-GDPR flow as I think that is the real decision point, with most having to handle GDPR style flows (voluntarily or by law) so I paid special attention to directing people how to do that so it is the same on both platforms using the platform-neutral javascript call

Words are easy to smush around though, do anything you like (or nothing at all) with the suggestion :-)

[github-actions]

🎉 This PR is included in version 4.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[thebergamo]

As you see, release done :)

[thebergamo]

Thank you very much @rnike for your PR and the commitment into it that was wonderful and @mikehardy for reviewing and providing valuable feedback on it as well.