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][NewArch] Add danger block for C++ TurboModule #4402

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

cipolleschi
Copy link
Contributor

We discovered that in App pure Cxx Turbomodules stopped working from version 0.76 of React Native.

We have the issue tracking it and a potential fix for it.

Given the Christmas break, we are not going to be able to ship a new release of React Native with the fix. Therefore we want to document this problem in the website.

Copy link

netlify bot commented Dec 20, 2024

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 207df01
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/6765540429cd930008dda910
😎 Deploy Preview https://deploy-preview-4402--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Dec 20, 2024

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 41ae98b
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/67655abeaca6400008e3fc97
😎 Deploy Preview https://deploy-preview-4402--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

docs/the-new-architecture/pure-cxx-modules.md Outdated Show resolved Hide resolved
Copy link

@timbocole timbocole left a comment

Choose a reason for hiding this comment

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

Thank You - this alleviates my concerns of a new dev getting very confused! 😍

@cipolleschi cipolleschi force-pushed the cipolleschi/add-danger-for-cxx-tm branch from 7fc4473 to 41ae98b Compare December 20, 2024 11:53
@cipolleschi cipolleschi merged commit a94002d into main Dec 20, 2024
4 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.

4 participants