feat: remove legacy arch support

Author: lposen

README.md

@@ -69,17 +69,12 @@ View the [API documentation](https://iterable-react-native-sdk.netlify.app).
 
 ## Architecture Support
 
-Iterable's React Native SDK supports [React Native's New
+Iterable's React Native SDK requires [React Native's New
 Architecture](https://reactnative.dev/architecture/landing-page), including
-TurboModules and Fabric.
+TurboModules and Fabric. Legacy Architecture is not supported.
 
-**IMPORTANT**: Iterable's React Native SDK supports React Native's Legacy Architecture, but it
-is no longer actively maintained.  Use at your own risk.
-
-Notes:
-
-- Ensure your app is configured for New Architecture per the React Native docs.
-- The example app in this repository is configured with New Architecture enabled.
+Ensure your app is configured for New Architecture per the React Native docs.
+The example app in this repository has New Architecture enabled.
 
 ## Beta Versions
 

android/build.gradle

@@ -15,16 +15,9 @@ buildscript {
   }
 }
 
-def isNewArchitectureEnabled() {
-  return rootProject.hasProperty("newArchEnabled") && rootProject.getProperty("newArchEnabled") == "true"
-}
-
 apply plugin: "com.android.library"
 apply plugin: "kotlin-android"
-
-if (isNewArchitectureEnabled()) {
-  apply plugin: "com.facebook.react"
-}
+apply plugin: "com.facebook.react"
 
 def getExtOrDefault(name) {
   return rootProject.ext.has(name) ? rootProject.ext.get(name) : project.properties["RNIterable_" + name]
@@ -59,7 +52,6 @@ android {
   defaultConfig {
     minSdkVersion getExtOrIntegerDefault("minSdkVersion")
     targetSdkVersion getExtOrIntegerDefault("targetSdkVersion")
-    buildConfigField("boolean", "IS_NEW_ARCHITECTURE_ENABLED", isNewArchitectureEnabled().toString())
   }
 
   buildFeatures {
@@ -81,16 +73,6 @@ android {
     targetCompatibility JavaVersion.VERSION_1_8
   }
 
-  sourceSets {
-    main {
-      if (isNewArchitectureEnabled()) {
-        java.srcDirs += ['src/newarch']
-      } else {
-        java.srcDirs += ['src/oldarch']
-      }
-    }
-  }
-
   // Add this to match the Iterable SDK group ID
   group = "com.iterable"
 }

…ewarch/java/com/RNIterableAPIModule.java …ble/reactnative/RNIterableAPIModule.javaandroid/src/newarch/java/com/RNIterableAPIModule.java renamed to android/src/main/java/com/iterable/reactnative/RNIterableAPIModule.java android/src/newarch/java/com/RNIterableAPIModule.java renamed to android/src/main/java/com/iterable/reactnative/RNIterableAPIModule.java

@@ -7,8 +7,6 @@
 import com.facebook.react.bridge.ReactApplicationContext;
 import com.facebook.react.bridge.ReadableArray;
 import com.facebook.react.bridge.ReadableMap;
-import com.iterable.iterableapi.AuthFailure;
-import com.iterable.iterableapi.IterableLogger;
 
 public class RNIterableAPIModule extends NativeRNIterableAPISpec {
   private final ReactApplicationContext reactContext;

android/src/main/java/com/iterable/reactnative/RNIterableAPIPackage.java

@@ -30,14 +30,13 @@ public ReactModuleInfoProvider getReactModuleInfoProvider() {
         @Override
         public Map<String, ReactModuleInfo> getReactModuleInfos() {
           Map<String, ReactModuleInfo> moduleInfos = new HashMap<>();
-          boolean isTurboModule = BuildConfig.IS_NEW_ARCHITECTURE_ENABLED;
           moduleInfos.put(RNIterableAPIModuleImpl.NAME, new ReactModuleInfo(
             RNIterableAPIModuleImpl.NAME,
             RNIterableAPIModuleImpl.NAME,
             false,  // canOverrideExistingModule
             false,  // needsEagerInit
             false,  // isCxxModule
-            isTurboModule    // isTurboModule
+            true    // isTurboModule
           ));
           return moduleInfos;
         }

android/src/oldarch/java/com/RNIterableAPIModule.java

@@ -1,18 +1,10 @@
 #import <Foundation/Foundation.h>
 #import <React/RCTEventEmitter.h>
+#import <React/RCTUtils.h>
+#import <React/RCTConvert.h>
+#import <React/RCTTurboModuleRegistry.h>
+#import <RNIterableAPISpec/RNIterableAPISpec.h>
 
-#if RCT_NEW_ARCH_ENABLED
-
-  #import <React/RCTUtils.h>
-  #import <React/RCTConvert.h>
-  #import <React/RCTTurboModuleRegistry.h>
-  #import <RNIterableAPISpec/RNIterableAPISpec.h>
 @interface RNIterableAPI : RCTEventEmitter <NativeRNIterableAPISpec>
 
-#else
-  #import <React/RCTBridgeModule.h>
-@interface RNIterableAPI : RCTEventEmitter <RCTBridgeModule>
-
-#endif
-
 @end