Skip to content

Conversation

@markdevocht
Copy link
Contributor

We were missing this one liner in our RNNAppDelegate
[RCTComponentViewFactory currentComponentViewFactory].thirdPartyFabricComponentsProvider = self;
which would assign a delegate that would take care of calling the auto-generated RCTThirdPartyComponentsProvider.mm to register 3rd party components.

@markdevocht markdevocht requested review from d4vidi and gosha212 July 23, 2025 11:09
@markdevocht markdevocht self-assigned this Jul 23, 2025
@markdevocht markdevocht marked this pull request as ready for review July 23, 2025 11:13
Copy link
Contributor

@gosha212 gosha212 left a comment

Choose a reason for hiding this comment

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

Amazing!
Lets just validate that we didn't miss anything in createRCTRootViewFactory
We don't need the code for bridge configuration

@markdevocht
Copy link
Contributor Author

All functionality in the RCTAppDelegate's createRCTRootViewFactory function, which we don't have, are related to the RCTBridge and I cannot find anything of these that could be translated to RCTHost.

@markdevocht markdevocht requested a review from gosha212 July 24, 2025 08:26
@markdevocht markdevocht merged commit d571fdc into master Jul 27, 2025
4 of 5 checks passed
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