Page MenuHomePhabricator
Differential D11663

[native] Stop using ReanimatedJSIModulePackage on Android
ClosedPublic
Actions

Authored by ashoat on Mon, Apr 15, 4:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 9:44 PM
Unknown Object (File)
Sat, Apr 27, 4:34 PM
Unknown Object (File)
Fri, Apr 26, 8:53 PM
Unknown Object (File)
Fri, Apr 26, 12:42 PM
Unknown Object (File)
Wed, Apr 24, 1:44 PM
Unknown Object (File)
Tue, Apr 23, 12:27 PM
Unknown Object (File)
Tue, Apr 23, 11:47 AM
Unknown Object (File)
Mon, Apr 22, 4:07 PM
View All 14 Files
Subscribers
None

Details

Reviewers
marcin
tomek
Commits
rCOMM13b3ce70a796: [native] Stop using ReanimatedJSIModulePackage on Android
Summary

While investigating ENG-7696, I noticed this log being printed:

[REANIMATED]: Since 2.5.0, Reanimated autoinstalls on Android - you can remove getJSIModulePackage() override in MainApplication.java.

It looks like it's coming from here, which confirms that ReanimatedJSIModulePackage is no longer doing anything and is deprecated.

This diff replaces our extension of ReanimatedJSIModulePackage with an implementation of its interface JSIModulePackage.

Depends on D11660

Test Plan
  1. Confirm that Reanimated still works in Android simulator
  2. Confirm that the log message is no longer printed

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
native/
android/
app/
src/
main/
java/
app/
comm/
android/
5 lines

Diff 39133

View Options

native/android/app/src/main/java/app/comm/android/CommCoreJSIModulePackage.java

Loading...