Remove timeout event

CHANGELOG.md

@@ -2,6 +2,12 @@
 
 The changelog for `Superwall`. Also see the [releases](https://github.com/superwall/Superwall-Android/releases) on GitHub.
 
+
+## 2.6.9
+
+## Deprecations
+- Deprecated `paywallWebviewLoad_timeout` - this event was causing confusion due to it's naming, leading to it being deprecated
+
 ## 2.6.8
 
 ### Enhancements

superwall/src/main/java/com/superwall/sdk/analytics/internal/trackable/TrackableSuperwallEvent.kt

@@ -712,10 +712,6 @@ sealed class InternalSuperwallEvent(
 
             object Fallback : State()
 
-            class Timeout(
-                val msg: String,
-            ) : State()
-
             class Complete : State()
         }
 
@@ -727,11 +723,6 @@ sealed class InternalSuperwallEvent(
                             paywallInfo,
                         )
 
-                    is PaywallWebviewLoad.State.Timeout ->
-                        SuperwallEvent.PaywallWebviewLoadTimeout(
-                            paywallInfo,
-                        )
-
                     is PaywallWebviewLoad.State.Fail ->
                         SuperwallEvent.PaywallWebviewLoadFail(
                             paywallInfo,
@@ -765,11 +756,6 @@ sealed class InternalSuperwallEvent(
                                 }.toTypedArray(),
                         )
 
-                    is State.Timeout ->
-                        mapOf(
-                            "error_message" to state.msg,
-                        )
-
                     else -> mapOf()
                 }
             val params = paywallInfo.eventParams() + extras

superwall/src/main/java/com/superwall/sdk/analytics/superwall/SuperwallEvent.kt

@@ -327,7 +327,7 @@ sealed class SuperwallEvent {
             get() = "paywallWebviewLoad_complete"
     }
 
-    // / When the loading of a paywall's website times out.
+    @Deprecated("Due to confusion this event was causing we're deprecating it's usage")
     data class PaywallWebviewLoadTimeout(
         val paywallInfo: PaywallInfo,
     ) : SuperwallEvent() {

superwall/src/main/java/com/superwall/sdk/analytics/superwall/SuperwallEvents.kt

@@ -34,6 +34,8 @@ enum class SuperwallEvents(
     PaywallWebviewLoadStart("paywallWebviewLoad_start"),
     PaywallWebviewLoadFail("paywallWebviewLoad_fail"),
     PaywallWebviewLoadComplete("paywallWebviewLoad_complete"),
+
+    @Deprecated("Due to confusion this event was causing we're deprecating it's usage")
     PaywallWebviewLoadTimeout("paywallWebviewLoad_timeout"),
     PaywallWebviewLoadFallback("paywallWebviewLoad_fallback"),
     PaywallResourceLoadFail("paywallResourceLoad_fail"),

superwall/src/test/java/com/superwall/sdk/analytics/internal/TrackingLogicTest.kt

@@ -436,30 +436,6 @@ class TrackingLogicTest {
             }
         }
 
-    @Test
-    fun `isNotDisabledVerboseEvent allows PaywallWebviewLoad Timeout even when verbose events disabled`() =
-        Given("PaywallWebviewLoad with Timeout state and verbose events disabled") {
-            val paywallInfo = PaywallInfo.empty()
-            val event =
-                InternalSuperwallEvent.PaywallWebviewLoad(
-                    state = InternalSuperwallEvent.PaywallWebviewLoad.State.Timeout("timeout"),
-                    paywallInfo = paywallInfo,
-                )
-
-            When("checking if event is not disabled verbose event") {
-                val result =
-                    TrackingLogic.isNotDisabledVerboseEvent(
-                        event,
-                        disableVerboseEvents = true,
-                        isSandbox = false,
-                    )
-
-                Then("it should return true because Timeout events are always allowed") {
-                    assertTrue(result)
-                }
-            }
-        }
-
     // ========== canTriggerPaywall Tests ==========
 
     @Test