Skip to content
This repository has been archived by the owner on Apr 2, 2021. It is now read-only.

Use RCTComponentEvent #487

Merged

Conversation

hramos
Copy link
Contributor

@hramos hramos commented Feb 27, 2019

Related to facebook/react-native#15894

Use RCTComponentEvent instead of the deprecated [sendInputEventWithName:body:] which will be removed when facebook/react-native#15894 lands.

This PR should only be merged once facebook/react-native#15894 has landed and made it to a stable release, since RCTComponentEvent does not currently exist in RN master.

@hramos hramos marked this pull request as ready for review February 28, 2019 01:33
Copy link
Contributor

@janicduplessis janicduplessis left a comment

Choose a reason for hiding this comment

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

🎉

@janicduplessis janicduplessis merged commit 3eb205e into facebookarchive:master Feb 28, 2019
@janicduplessis
Copy link
Contributor

This PR should only be merged once facebook/react-native#15894 has landed and made it to a stable release, since RCTComponentEvent does not currently exist in RN master

Oups 🤦‍♂️

I thought you wanted to land this before [facebook/react-native#15894], might want to revert unless we're not planning to do a release of fbsdk for a while.

@hramos
Copy link
Contributor Author

hramos commented Feb 28, 2019

No problem. I’m looking into cutting a release soon, but if that PR gets merged and picked into 0.59 the timing might work out alright.

@hramos hramos deleted the remove-deprecated-event branch March 1, 2019 15:25
redcancode added a commit to redcancode/react-native-fbsdk that referenced this pull request Jun 15, 2019
redcancode added a commit to redcancode/react-native-fbsdk that referenced this pull request Jun 17, 2019
mdaines-root added a commit to Root-App/react-native-fbsdk that referenced this pull request Oct 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants