Fix isAutomaticConnectionEnabled

kean · Oct 11, 2024 · 4aa9738 · 4aa9738
1 parent c103fa2
commit 4aa9738
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Pulse 5.x
 
+## Pulse 5.1.2
+
+- Fix an issue with `isAutomaticConnectionEnabled` not starting the browser the first time it's enabled
+
 ## Pulse 5.1.1
 
 *Sep 19, 2024*

diff --git a/Demo/Sources/iOS/Pulse_Demo_iOSApp.swift b/Demo/Sources/iOS/Pulse_Demo_iOSApp.swift
@@ -26,7 +26,7 @@ private final class AppViewModel: ObservableObject {
         // This code registers the store with the `RemoteLogger` (important!)
         LoggerStore.shared = .demo
 
-        RemoteLogger.shared.isAutomaticConnectionEnabled = true
+        // RemoteLogger.shared.isAutomaticConnectionEnabled = true
 
         // NetworkLogger.enableProxy()
 

diff --git a/Sources/Pulse/RemoteLogger/RemoteLogger.swift b/Sources/Pulse/RemoteLogger/RemoteLogger.swift
@@ -111,7 +111,8 @@ public final class RemoteLogger: ObservableObject, RemoteLoggerConnectionDelegat
     /// requires Bluetooth, it will trigger a prompt on the first app run.
     public var isAutomaticConnectionEnabled = false {
         didSet {
-            isEnabled = true
+            guard isAutomaticConnectionEnabled else { return }
+            enable()
         }
     }