Support for Turbo Module/new RN architecture on Android

[anders-friis-topdk]

Description

When running on Android with newArchEnabled=true in gradle.properties, RiveReactNativeView shows on screen that: 'RiveReactNativeView' is not Fabric compatible yet.

I am trying to show a very simple Rive animation.

Versions:

• react-native: 0.72.3
• rive-react-native: 4.1.2
• Gradle properties (android/gradle.properties):
  • newArchEnabled=true
  • hermesEnabled=true
  • android.useAndroidX=true
  • android.enableJetifier=true
  • reactNativeArchitectures=armeabi-v7a,arm64-v8a,x86,x86_64
  • FLIPPER_VERSION=0.182.0
• OS: Android minSdkVersion = 21

Expected behavior

• Fabric/newArchEnabled should be supported on Android

Could be good to add status for the library to the '3rd party libraries - updates and requests' list here:
reactwg/react-native-new-architecture#6

[HayesGordon]

Hi @anders-friis-topdk thanks for reporting. We just did a release that should resolve this, can you try with the latest v5.0.0?

[anders-friis-topdk]

Hi @HayesGordon, thank you for a quick response and release!
I just tried a couple of times with the new v5.0.0 but I couldn't get it to work.
So I created a new test project and I ended up with the same result.

Here is what I did in the new test project:

• Create new RN project with v. 0.72.3:
  • npx react-native@latest init RiveFabricTest
• Setting in gradle.properties:
  • newArchEnabled=true
• Add this in app/build.gradle:
  • implementation platform('org.jetbrains.kotlin:kotlin-bom:1.8.0')
• Add rive:
  • rive-react-native": "^5.0.0"
• Add HTTP rive component from example project in App.tsx

I end up with the same error view:

Here is my new test project:
RiveFabricTest.zip

[HayesGordon]

Hi @anders-friis-topdk thanks for taking a look into this.

Sorry I wasted your time. I was under the wrong impression for what was required for Fabric to work - it'll involve a few extra steps, and we also need to make sure it remains compatible with the old architecture.

We'll investigate adding support for this and I'll update here.

[anders-friis-topdk]

Any news on this issue @HayesGordon ?

[AamirHafiez]

@HayesGordon Any update on this?

[BitBarrel]

Same problem here.

[anders-friis-topdk]

Long time passed since last update on this one. This is blocking our project from using new architecture on Android so it is quite important to us.
Any news @HayesGordon ?

[patrikduksin]

with new arch doesn't work on ios either, version 6.2.3

[hyochan]

The React Native ecosystem is evolving, with a shift towards a new architecture. To stay current and ensure our library remains useful, it is essential that we adapt to this new architecture.

We hope for timely updates to support this transition effectively.

[tslater]

@HayesGordon Would the rive team accept PRs for this? Not sure if we have capacity, but this is climbing in priority for us and want to understand the possibilities.

[HayesGordon]

Hi @tslater, we love contributions and would be happy to accept a PR

[tslater]

@HayesGordon Thanks! We decide to start with a smaller contribution to warm up. Do you mind taking a look at this PR and giving any feedback?

#278

[zownz-tech]

Is this still an issue for v8?

[jsd-within]

Is this still an issue for v8?

I don't know if it is perfect yet, but I have 8.3 running in a Fabric app. I just had to add:

import {registerCallableModule} from 'react-native';
...
registerCallableModule('RCTEventEmitter', {
  receiveEvent: () => {
    console.log('received event');
  },
});