Skip to content

Commit

Permalink
Remove "Thank You" prompt (#2762)
Browse files Browse the repository at this point in the history
  • Loading branch information
samsymons authored Apr 18, 2024
1 parent dd16758 commit 499c595
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 36 deletions.
2 changes: 0 additions & 2 deletions Core/PixelEvent.swift
Original file line number Diff line number Diff line change
Expand Up @@ -590,7 +590,6 @@ extension Pixel {
case privacyProSubscriptionManagementPlanBilling
case privacyProSubscriptionManagementRemoval
case privacyProFeatureEnabled
case privacyProPromotionDialogShownVPN
case privacyProVPNAccessRevokedDialogShown
case privacyProVPNBetaStoppedWhenPrivacyProEnabled
case privacyProTransactionProgressNotHiddenAfter60s
Expand Down Expand Up @@ -1256,7 +1255,6 @@ extension Pixel.Event {

// Launch
case .privacyProFeatureEnabled: return "m_privacy-pro_feature_enabled"
case .privacyProPromotionDialogShownVPN: return "m_privacy-pro_promotion-dialog_shown_vpn"
case .privacyProVPNAccessRevokedDialogShown: return "m_privacy-pro_vpn-access-revoked-dialog_shown"
case .privacyProVPNBetaStoppedWhenPrivacyProEnabled: return "m_privacy-pro_vpn-beta-stopped-when-privacy-pro-enabled"

Expand Down
10 changes: 0 additions & 10 deletions DuckDuckGo/AppDelegate.swift
Original file line number Diff line number Diff line change
Expand Up @@ -424,14 +424,6 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
)
presenter.showEntitlementNotification()
}

private func presentVPNEarlyAccessOverAlert() {
let alertController = CriticalAlerts.makeVPNEarlyAccessOverAlert()
window?.rootViewController?.present(alertController, animated: true) { [weak self] in
DailyPixel.fireDailyAndCount(pixel: .privacyProPromotionDialogShownVPN)
self?.tunnelDefaults.vpnEarlyAccessOverAlertAlreadyShown = true
}
}
#endif

private func cleanUpMacPromoExperiment2() {
Expand Down Expand Up @@ -543,8 +535,6 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
}

if vpnFeatureVisibility.shouldShowThankYouMessaging() && !tunnelDefaults.vpnEarlyAccessOverAlertAlreadyShown {
presentVPNEarlyAccessOverAlert()

Task {
await self.stopAndRemoveVPN(with: "thank-you-dialog")
}
Expand Down
11 changes: 0 additions & 11 deletions DuckDuckGo/CriticalAlerts.swift
Original file line number Diff line number Diff line change
Expand Up @@ -83,15 +83,4 @@ struct CriticalAlerts {
return alertController
}

static func makeVPNEarlyAccessOverAlert() -> UIAlertController {
let alertController = UIAlertController(title: UserText.vpnEarlyAccessOverAlertTitle,
message: UserText.vpnEarlyAccessOverAlertMessage,
preferredStyle: .alert)

let closeButton = UIAlertAction(title: UserText.vpnEarlyAccessOverAlertAction, style: .cancel)

alertController.addAction(closeButton)
return alertController
}

}
4 changes: 0 additions & 4 deletions DuckDuckGo/UserText.swift
Original file line number Diff line number Diff line change
Expand Up @@ -674,10 +674,6 @@ In addition to the details entered into this form, your app issue report will co
static let vpnAccessRevokedAlertActionSubscribe = NSLocalizedString("vpn.access-revoked.alert.action.subscribe", value: "Subscribe", comment: "Primary action for the alert when the subscription expires")
static let vpnAccessRevokedAlertActionCancel = NSLocalizedString("vpn.access-revoked.alert.action.cancel", value: "Dismiss", comment: "Cancel action for the alert when the subscription expires")

static let vpnEarlyAccessOverAlertTitle = NSLocalizedString("vpn.early-access.over.alert.title", value: "DuckDuckGo VPN early access is over", comment: "Alert title for the alert when the early access period is over")
static let vpnEarlyAccessOverAlertMessage = NSLocalizedString("vpn.early-access.over.alert.message", value: "Thank you for being a tester! To continue using the VPN, subscribe to DuckDuckGo Privacy Pro and get 40% off with promo code THANKYOU\n\nOffer redeemable for a limited time only in the desktop version of the DuckDuckGo browser by U.S. testers who install from duckduckgo.com/app", comment: "Alert message for the alert when the early access period is over")
static let vpnEarlyAccessOverAlertAction = NSLocalizedString("vpn.early-access.over.alert.action", value: "OK", comment: "Alert action for the alert when the early access period is over")

// MARK: Notifications

public static let macWaitlistAvailableNotificationTitle = NSLocalizedString("mac-waitlist.available.notification.title", value: "DuckDuckGo for Mac is ready!", comment: "Title for the macOS waitlist notification")
Expand Down
9 changes: 0 additions & 9 deletions DuckDuckGo/en.lproj/Localizable.strings
Original file line number Diff line number Diff line change
Expand Up @@ -2435,15 +2435,6 @@ But if you *do* want a peek under the hood, you can find more information about
/* Alert title for the alert when the Privacy Pro subscription expires */
"vpn.access-revoked.alert.title" = "VPN disconnected due to expired subscription";

/* Alert action for the alert when the early access period is over */
"vpn.early-access.over.alert.action" = "OK";

/* Alert message for the alert when the early access period is over */
"vpn.early-access.over.alert.message" = "Thank you for being a tester! To continue using the VPN, subscribe to DuckDuckGo Privacy Pro and get 40% off with promo code THANKYOU\n\nOffer redeemable for a limited time only in the desktop version of the DuckDuckGo browser by U.S. testers who install from duckduckgo.com/app";

/* Alert title for the alert when the early access period is over */
"vpn.early-access.over.alert.title" = "DuckDuckGo VPN early access is over";

/* Title for the Cancel button of the VPN feedback form */
"vpn.feedback-form.button.cancel" = "Cancel";

Expand Down

0 comments on commit 499c595

Please sign in to comment.