♻️ Update debugger to ping healthcheck only once opened

appcues · Jun 6, 2022 · 0ce414d · 0ce414d
1 parent f148d0d
commit 0ce414d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/Sources/AppcuesKit/Presentation/Debugger/Panel/DebugViewModel.swift b/Sources/AppcuesKit/Presentation/Debugger/Panel/DebugViewModel.swift
@@ -78,10 +78,7 @@ internal class DebugViewModel: ObservableObject {
         self.isAnonymous = isAnonymous
         self.userIdentified = !currentUserID.isEmpty
 
-        // Initial connectivity check
-        ping()
-
-        connectedStatus.action = Action(symbolName: "arrow.triangle.2.circlepath") { self.ping() }
+        connectedStatus.action = Action(symbolName: "arrow.triangle.2.circlepath") { [weak self] in self?.ping() }
     }
 
     func reset() {

diff --git a/Sources/AppcuesKit/Presentation/Debugger/UIDebugger.swift b/Sources/AppcuesKit/Presentation/Debugger/UIDebugger.swift
@@ -102,8 +102,13 @@ internal class UIDebugger: UIDebugging {
 @available(iOS 13.0, *)
 extension UIDebugger: DebugViewDelegate {
     func debugView(did event: DebugView.Event) {
-        if case .hide = event {
+        switch event {
+        case .hide:
             hide()
+        case .open:
+            viewModel.ping()
+        case .show, .close, .reposition:
+            break
         }
     }
 }