Config now saved statically inside Iterable

Context: Android Back Button destroys MainActivity while minimizing the app. Thus front end code is removed from memory. However, it still keeps old config handlers in memory which can hinder rendering view and navigating once we wake app on tap of push notification.

This fix will allow all event handlers to use the latest and available IterableConfig as we will statically store the config when initialzing and replacing the static old one.

Author: Ayyanchira

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

@@ -372,8 +372,8 @@ public void run() {
     @ReactMethod
     public void wakeApp() {
         Intent launcherIntent = getMainActivityIntent(reactContext);
-        launcherIntent.setFlags(Intent.FLAG_FROM_BACKGROUND);
-        if (launcherIntent.resolveActivity(getReactApplicationContext().getPackageManager()) != null) {
+        launcherIntent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK | Intent.FLAG_ACTIVITY_CLEAR_TOP | Intent.FLAG_ACTIVITY_SINGLE_TOP);
+        if (launcherIntent.resolveActivity(reactContext.getPackageManager()) != null) {
             reactContext.startActivity(launcherIntent);
         }
     }

ts/Iterable.ts

@@ -199,6 +199,12 @@ class Iterable {
   */
   static inAppManager = new IterableInAppManager()
 
+  
+  /**
+   * savedConfig instance.
+   */ 
+  static savedConfig: IterableConfig
+
   /**
   * 
   * @param {string} apiKey 
@@ -207,7 +213,8 @@ class Iterable {
   static initialize(apiKey: string, config: IterableConfig = new IterableConfig()): Promise<boolean> {
     console.log("initialize: " + apiKey);
 
-    this.setupEventHandlers(config)
+    Iterable.savedConfig = config
+    this.setupEventHandlers()
     const version = this.getVersionFromPackageJson()
 
     return RNIterableAPI.initializeWithApiKey(apiKey, config.toDict(), version)
@@ -220,7 +227,8 @@ class Iterable {
   static initialize2(apiKey: string, config: IterableConfig = new IterableConfig(), apiEndPoint: string): Promise<boolean> {
     console.log("initialize2: " + apiKey);
 
-    this.setupEventHandlers(config)
+    Iterable.savedConfig = config
+    this.setupEventHandlers()
     const version = this.getVersionFromPackageJson()
 
     return RNIterableAPI.initialize2WithApiKey(apiKey, config.toDict(), version, apiEndPoint)
@@ -314,8 +322,10 @@ class Iterable {
   }
 
   static wakeApp() {
-    console.log('Attempting to wake the app')
-    RNIterableAPI.wakeApp();
+    if (Platform.OS === "android") {
+      console.log('Attempting to wake the app')
+      RNIterableAPI.wakeApp();
+    }
   }
 
   /**
@@ -431,53 +441,52 @@ class Iterable {
   }
 
   // PRIVATE
-  private static setupEventHandlers(config: IterableConfig) {
-    if (config.urlHandler) {
+  private static setupEventHandlers() {
+     //Remove all listeners to avoid duplicate listeners
+     RNEventEmitter.removeAllListeners(EventName.handleUrlCalled)
+     RNEventEmitter.removeAllListeners(EventName.handleInAppCalled)
+     RNEventEmitter.removeAllListeners(EventName.handleCustomActionCalled)
+     RNEventEmitter.removeAllListeners(EventName.handleAuthCalled)
+
+    if (Iterable.savedConfig.urlHandler) {
       RNEventEmitter.addListener(
         EventName.handleUrlCalled,
         (dict) => {
           const url = dict["url"]
           const context = IterableActionContext.fromDict(dict["context"])
-          if (Platform.OS === "android") {
-            Iterable.wakeApp()
-            //Give enough time for Activity to wake up.
-            setTimeout(() => {
-              callUrlHandler(url, context)
-            }, 1000)
-          } else {
-            callUrlHandler(url, context)
-          }
+          Iterable.wakeApp()
+          callUrlHandler(url, context)
         }
       )
     }
 
-    if (config.customActionHandler) {
+    if (Iterable.savedConfig.customActionHandler) {
       RNEventEmitter.addListener(
         EventName.handleCustomActionCalled,
         (dict) => {
           const action = IterableAction.fromDict(dict["action"])
           const context = IterableActionContext.fromDict(dict["context"])
-          config.customActionHandler!(action, context)
+          Iterable.savedConfig.customActionHandler!(action, context)
         }
       )
     }
 
-    if (config.inAppHandler) {
+    if (Iterable.savedConfig.inAppHandler) {
       RNEventEmitter.addListener(
         EventName.handleInAppCalled,
         (messageDict) => {
           const message = IterableInAppMessage.fromDict(messageDict)
-          const result = config.inAppHandler!(message)
+          const result = Iterable.savedConfig.inAppHandler!(message)
           RNIterableAPI.setInAppShowResponse(result)
         }
       )
     }
 
-    if (config.authHandler) {
+    if (Iterable.savedConfig.authHandler) {
       RNEventEmitter.addListener(
         EventName.handleAuthCalled,
         () => {
-          config.authHandler!()
+          Iterable.savedConfig.authHandler!()
             .then(authToken => {
               RNIterableAPI.passAlongAuthToken(authToken)
             })
@@ -486,7 +495,7 @@ class Iterable {
     }
 
     function callUrlHandler(url: any, context: IterableActionContext) {
-      if (config.urlHandler!(url, context) == false) {
+      if (Iterable.savedConfig.urlHandler!(url, context) == false) {
         Linking.canOpenURL(url)
           .then(canOpen => {
             if (canOpen) { Linking.openURL(url) }