Removed VPN waitlist beta pixels (#2686)

Task/Issue URL: https://app.asana.com/0/0/1207008939337649/f macOS PR: duckduckgo/macos-browser#2555 ## Description Removes all VPN waitlist beta pixels.
duckduckgo · Apr 8, 2024 · 75ae662 · 75ae662
1 parent 0f9d596
commit 75ae662
Show file tree

Hide file tree

Showing 7 changed files with 0 additions and 78 deletions.
diff --git a/Core/PixelEvent.swift b/Core/PixelEvent.swift
@@ -345,15 +345,6 @@ extension Pixel {
 
         case networkProtectionUnhandledError
 
-        case networkProtectionWaitlistUserActive
-        case networkProtectionSettingsRowDisplayed
-        case networkProtectionWaitlistIntroScreenDisplayed
-        case networkProtectionWaitlistTermsDisplayed
-        case networkProtectionWaitlistTermsAccepted
-        case networkProtectionWaitlistNotificationShown
-        case networkProtectionWaitlistNotificationLaunched
-        case networkProtectionWaitlistRetriedInviteCodeRedemption
-
         case networkProtectionGeoswitchingOpened
         case networkProtectionGeoswitchingSetNearest
         case networkProtectionGeoswitchingSetCustom
@@ -929,15 +920,6 @@ extension Pixel.Event {
         case .networkProtectionMemoryCritical: return "m_netp_vpn_memory_critical"
         case .networkProtectionUnhandledError: return "m_netp_unhandled_error"
 
-        case .networkProtectionWaitlistUserActive: return "m_netp_waitlist_user_active"
-        case .networkProtectionSettingsRowDisplayed: return "m_netp_waitlist_settings_entry_viewed"
-        case .networkProtectionWaitlistIntroScreenDisplayed: return "m_netp_waitlist_intro_screen_viewed"
-        case .networkProtectionWaitlistTermsDisplayed: return "m_netp_waitlist_terms_viewed"
-        case .networkProtectionWaitlistTermsAccepted: return "m_netp_waitlist_terms_accepted"
-        case .networkProtectionWaitlistNotificationShown: return "m_netp_waitlist_notification_shown"
-        case .networkProtectionWaitlistNotificationLaunched: return "m_netp_waitlist_notification_launched"
-        case .networkProtectionWaitlistRetriedInviteCodeRedemption: return "m_netp_waitlist_retried_invite_code_redemption"
-
         case .networkProtectionGeoswitchingOpened: return "m_netp_imp_geoswitching"
         case .networkProtectionGeoswitchingSetNearest: return "m_netp_ev_geoswitching_set_nearest"
         case .networkProtectionGeoswitchingSetCustom: return "m_netp_ev_geoswitching_set_custom"

diff --git a/DuckDuckGo/AppDelegate+Waitlists.swift b/DuckDuckGo/AppDelegate+Waitlists.swift
@@ -46,9 +46,6 @@ extension AppDelegate {
 #if NETWORK_PROTECTION
     private func checkNetworkProtectionWaitlist() {
         let accessController = NetworkProtectionAccessController()
-        if accessController.isPotentialOrCurrentWaitlistUser {
-            DailyPixel.fire(pixel: .networkProtectionWaitlistUserActive)
-        }
 
         VPNWaitlist.shared.fetchInviteCodeIfAvailable { [weak self] error in
             guard error == nil else {
@@ -70,8 +67,6 @@ extension AppDelegate {
             do {
                 try await NetworkProtectionCodeRedemptionCoordinator().redeem(inviteCode)
                 VPNWaitlist.shared.sendInviteCodeAvailableNotification()
-
-                DailyPixel.fireDailyAndCount(pixel: .networkProtectionWaitlistNotificationShown)
             } catch {}
         }
     }

diff --git a/DuckDuckGo/AppDelegate.swift b/DuckDuckGo/AppDelegate.swift
@@ -1010,7 +1010,6 @@ extension AppDelegate: UNUserNotificationCenterDelegate {
 
             if vpnFeatureVisibility.shouldKeepVPNAccessViaWaitlist(), identifier == VPNWaitlist.notificationIdentifier {
                 presentNetworkProtectionWaitlistModal()
-                DailyPixel.fire(pixel: .networkProtectionWaitlistNotificationLaunched)
             }
 
 #endif

diff --git a/DuckDuckGo/NetworkProtectionAccessController.swift b/DuckDuckGo/NetworkProtectionAccessController.swift
@@ -70,15 +70,6 @@ struct NetworkProtectionAccessController: NetworkProtectionAccess {
         return (regionCode ?? "US") == "US"
     }
 
-    var isPotentialOrCurrentWaitlistUser: Bool {
-        switch self.networkProtectionAccessType() {
-        case .none, .inviteCodeInvited:
-            return false
-        case .waitlistAvailable, .waitlistJoined, .waitlistInvitedPendingTermsAcceptance, .waitlistInvited:
-            return true
-        }
-    }
-
     init(
         networkProtectionActivation: NetworkProtectionFeatureActivation = NetworkProtectionKeychainTokenStore(),
         networkProtectionWaitlistStorage: WaitlistStorage = WaitlistKeychainStore(waitlistIdentifier: VPNWaitlist.identifier),

diff --git a/DuckDuckGo/VPNWaitlistTermsAndConditionsViewController.swift b/DuckDuckGo/VPNWaitlistTermsAndConditionsViewController.swift
@@ -40,17 +40,13 @@ final class VPNWaitlistTermsAndConditionsViewController: UIViewController {
 
         title = UserText.netPNavTitle
         addHostingControllerToViewHierarchy()
-
-        DailyPixel.fire(pixel: .networkProtectionWaitlistTermsDisplayed)
     }
 
     private func addHostingControllerToViewHierarchy() {
         let waitlistView = VPNWaitlistPrivacyPolicyView { _ in
             var termsAndConditionsStore = NetworkProtectionTermsAndConditionsUserDefaultsStore()
             termsAndConditionsStore.networkProtectionWaitlistTermsAndConditionsAccepted = true
 
-            DailyPixel.fire(pixel: .networkProtectionWaitlistTermsAccepted)
-
             self.navigationController?.popToRootViewController(animated: true)
             let networkProtectionViewController = NetworkProtectionRootViewController()
             self.navigationController?.pushViewController(networkProtectionViewController, animated: true)

diff --git a/DuckDuckGo/VPNWaitlistViewController.swift b/DuckDuckGo/VPNWaitlistViewController.swift
@@ -68,10 +68,6 @@ final class VPNWaitlistViewController: UIViewController {
     private func updateViewState() {
         Task {
             await self.viewModel.updateViewState()
-
-            if self.viewModel.viewState == .notJoinedQueue {
-                DailyPixel.fire(pixel: .networkProtectionWaitlistIntroScreenDisplayed)
-            }
         }
     }
 

diff --git a/DuckDuckGoTests/NetworkProtectionAccessControllerTests.swift b/DuckDuckGoTests/NetworkProtectionAccessControllerTests.swift
@@ -117,43 +117,6 @@ final class NetworkProtectionAccessControllerTests: XCTestCase {
         XCTAssertEqual(controller.networkProtectionAccessType(), .waitlistInvited)
     }
 
-    func testWhenUserHasWaitlistAccess_ThenWaitlistUserCheckIsTrue() {
-        let controller = createMockAccessController(
-            isInternalUser: true,
-            featureActivated: true,
-            termsAccepted: true,
-            featureFlagsEnabled: true,
-            hasJoinedWaitlist: true,
-            hasBeenInvited: true
-        )
-
-        XCTAssertTrue(controller.isPotentialOrCurrentWaitlistUser)
-    }
-
-    func testWhenUserDoesNotHaveWaitlistAccess_ThenWaitlistUserCheckIsFalse() {
-        let controller = createMockAccessController(
-            featureActivated: false,
-            termsAccepted: false,
-            featureFlagsEnabled: false,
-            hasJoinedWaitlist: false,
-            hasBeenInvited: false
-        )
-
-        XCTAssertFalse(controller.isPotentialOrCurrentWaitlistUser)
-    }
-
-    func testWhenUserIsInternal_ThenWaitlistUserCheckIsFalse() {
-        let controller = createMockAccessController(
-            featureActivated: true,
-            termsAccepted: false,
-            featureFlagsEnabled: false,
-            hasJoinedWaitlist: false,
-            hasBeenInvited: false
-        )
-
-        XCTAssertFalse(controller.isPotentialOrCurrentWaitlistUser)
-    }
-
     // MARK: - Mock Creation
 
     private func createMockAccessController(
-Original file line number
+Diff line change
@@ Expand Up / @@ -1010,7 +1010,6 @@ extension AppDelegate: UNUserNotificationCenterDelegate { @@
                 if vpnFeatureVisibility.shouldKeepVPNAccessViaWaitlist(), identifier == VPNWaitlist.notificationIdentifier {
                     presentNetworkProtectionWaitlistModal()
-                    DailyPixel.fire(pixel: .networkProtectionWaitlistNotificationLaunched)
                 }
     #endif
@@ Expand Down @@