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

[0.75] Add support for handling com.facebook.react.bridge.Dynamic as parameter for TurboModules #46066

Merged
merged 1 commit into from
Aug 19, 2024

Commits on Aug 16, 2024

  1. Add support for handling com.facebook.react.bridge.Dynamic as param…

    …eter type in TurboModules (#45944)
    
    Summary:
    Pull Request resolved: #45944
    
    This diff adds support having (Legacy) Native Modules with functions with parameters of type `Dynamic`.
    This is currently blocking some libraries making it harder for them to migrate to New Architecture.
    
    I've implemented it by adding a `DynamicNative` implementation of `Dynamic` which holds a reference of
    the payload as a `folly::dynamic`.
    
    Changelog:
    [Android] [Added] - Add support for handling `com.facebook.react.bridge.Dynamic` as parameter type in TurboModules
    
    Reviewed By: mdvacca, cipolleschi
    
    Differential Revision: D60966684
    
    fbshipit-source-id: 2e63bc53ede5277a9c12f1b19f05f6099f5f35f9
    cortinico committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    7716b58 View commit details
    Browse the repository at this point in the history