Skip to content

Commit

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

Summary:
Pull Request resolved: facebook#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: 9c863f25086c459ed374bd0c57868e4bbdbde196
  • Loading branch information
mdvacca committed Sep 13, 2023
1 parent f7b961c commit ae9c3f3
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 ae9c3f3

Please sign in to comment.