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

Use RCTComponentEvent - #487

Merged
janicduplessis merged 1 commit into
facebookarchive:masterfrom
hramos:remove-deprecated-event
Feb 28, 2019
Merged

Use RCTComponentEvent#487
janicduplessis merged 1 commit into
facebookarchive:masterfrom
hramos:remove-deprecated-event

Conversation

@hramos

@hramos hramos commented Feb 27, 2019

Copy link
Copy Markdown
Contributor

Related to react/react-native#15894

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

This PR should only be merged once react/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

@janicduplessis janicduplessis left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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 [react/react-native#15894], might want to revert unless we're not planning to do a release of fbsdk for a while.

@hramos

hramos commented Feb 28, 2019

Copy link
Copy Markdown
Contributor Author

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