Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RN findNodeHandle no longer adds props directly to read-only owner #10520

Conversation

bvaughn
Copy link
Contributor

@bvaughn bvaughn commented Aug 23, 2017

Resolves #10518.

Builds on top of #10511 to avoid causing merge conflicts for #10511 or #10517.

screen shot 2017-08-23 at 8 16 48 am

@bvaughn bvaughn requested review from gaearon and acdlite August 23, 2017 15:19
@bvaughn bvaughn force-pushed the fix-rn-render-calls-to-findnodehandle branch from 6e82ddd to 5016cd8 Compare August 23, 2017 15:37
@bvaughn bvaughn force-pushed the fix-rn-render-calls-to-findnodehandle branch from 5016cd8 to e3736ef Compare August 23, 2017 15:44
Copy link
Collaborator

@acdlite acdlite left a comment

Choose a reason for hiding this comment

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

LGTM

@bvaughn bvaughn merged commit 55fc7a7 into facebook:delete-rn-stack Aug 23, 2017
@bvaughn bvaughn deleted the fix-rn-render-calls-to-findnodehandle branch August 23, 2017 18:30
@bvaughn bvaughn restored the fix-rn-render-calls-to-findnodehandle branch August 23, 2017 18:31
@bvaughn bvaughn deleted the fix-rn-render-calls-to-findnodehandle branch August 23, 2017 18:31
@bvaughn bvaughn restored the fix-rn-render-calls-to-findnodehandle branch August 23, 2017 18:32
bvaughn added a commit that referenced this pull request Aug 23, 2017
* Removed createReactNativeComponentClassStack and renamed createReactNativeComponentClassFiber => createReactNativeComponentClass

* Removed findNumericNodeHandleStack and renamed findNumericNodeHandleFiber => findNumericNodeHandle

* Renamed ReactNativeFiberEntry => ReactNativeEntry

* Removed all references to ReactNativeFeatureFlags and RN stack

* Removed severl RN modules that are no longer used

* Renamed ReactNativeEntry => ReactNativeFiberEntry for now. We'll probably remove 'fiber' references later

* Update build results json

* Deleted snapshot

* Re-add accidentally deleted test

* Remove now-unnecessary hack in test

* Fix lint

* RN findNodeHandle no longer adds props directly to read-only owner (#10520)

* Added ReactNativeMount-test snapshot for 'renders and reorders children' test
@bvaughn bvaughn deleted the fix-rn-render-calls-to-findnodehandle branch August 30, 2017 21:49
@bvaughn bvaughn mentioned this pull request Sep 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants