Skip to content

Commit

Permalink
Deprecate ReactPackage.createNativeModules recommending using getModu…
Browse files Browse the repository at this point in the history
…le instead (#39339)

Summary:
Pull Request resolved: #39339

Deprecating createNativeModules method from ReactPackage interface recommending using getModule instead in the new architecture of React Native

changelog: [Android][Changed] Deprecating createNativeModules method from ReactPackage interface recommending using getModule instead in the new architecture of React Native

Reviewed By: cortinico

Differential Revision: D48992719

fbshipit-source-id: 13e9345dffd2d1216eb20679492dff6306ca3348
  • Loading branch information
mdvacca committed Sep 8, 2023
1 parent 94bc876 commit c4f7a37
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
import com.facebook.infer.annotation.Nullsafe;
import com.facebook.react.bridge.NativeModule;
import com.facebook.react.bridge.ReactApplicationContext;
import com.facebook.react.common.annotations.DeprecatedInNewArchitecture;
import com.facebook.react.uimanager.UIManagerModule;
import com.facebook.react.uimanager.ViewManager;
import java.util.List;
Expand All @@ -37,9 +38,11 @@ public interface ReactPackage {

/**
* @param reactContext react application context that can be used to create modules
* @return list of native modules to register with the newly created catalyst instance
* @return list of native modules to register with the newly created catalyst instance This method
* is deprecated in the new Architecture of React Native.
*/
@NonNull
@DeprecatedInNewArchitecture
List<NativeModule> createNativeModules(@NonNull ReactApplicationContext reactContext);

/** @return a list of view managers that should be registered with {@link UIManagerModule} */
Expand Down

0 comments on commit c4f7a37

Please sign in to comment.