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

Delete bridge/JSIModule* #42115

Closed

Commits on Apr 19, 2024

  1. Delete deprecated JSIModule methods (facebook#42115)

    Summary:
    
    React Native Android had a concept called JSIModules, which iOS doesn't have. The JSIModule concept was introduced in the early stages of the Fabric project to represent modules that interact with JS through JSI and they are not NativeModules.
    
    In the new architecture this concept is not really necessary and these interfaces were only used to initialize and destroy the Fabric renderer and TurboModule Manager in react native core. Bridgeless mode doesn’t use JSIModule anymore. Also, it has an explicit list of supported JSI module types, so is not open for extension.
    In order to simplify RN concepts and reduce confusion with TurboModules, which also "use JSI", deleting everything related to JSIModule. This was already deprecated in 0.74.0.
    
    Please use ReactInstanceEventListener to subscribe for react instance events instead of getJSIModule() and we recommend using TurboModules instead of JSIModules.
    
    Changelog:
    [General][Breaking] Delete JSIModule
    
    Reviewed By: javache, cortinico
    
    Differential Revision: D49597702
    arushikesarwani94 authored and facebook-github-bot committed Apr 19, 2024
    Configuration menu
    Copy the full SHA
    f61c602 View commit details
    Browse the repository at this point in the history