Fix #8431: Copy mistake in confirmation message after resetting VPN c…

…onfig (#8443)
brave · Nov 20, 2023 · eb998ea · eb998ea
1 parent 97e2bb9
commit eb998ea
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 6 deletions.
diff --git a/Sources/BraveStrings/BraveStrings.swift b/Sources/BraveStrings/BraveStrings.swift
@@ -2745,6 +2745,11 @@ extension Strings {
         value: "Manage Subscription",
         comment: "Button to manage your VPN subscription")
 
+    public static let settingsRedeemOfferCode =
+      NSLocalizedString("vpn.settingsRedeemOfferCode", tableName: "BraveShared", bundle: .module,
+        value: "Redeem Offer Code",
+        comment: "Button to redeem offer code subscription")
+
     public static let settingsLinkReceipt =
       NSLocalizedString("vpn.settingsLinkReceipt", tableName: "BraveShared", bundle: .module,
         value: "Link purchase to your Brave account",
@@ -2977,22 +2982,27 @@ extension Strings {
 
     public static let resetVPNErrorTitle =
       NSLocalizedString("vpn.resetVPNErrorTitle", tableName: "BraveShared", bundle: .module,
-        value: "Error",
+        value: "Reset Failed",
         comment: "Title for error message when vpn configuration reset fails.")
 
     public static let resetVPNErrorBody =
       NSLocalizedString("vpn.resetVPNErrorBody", tableName: "BraveShared", bundle: .module,
-        value: "Failed to reset vpn configuration, please try again later.",
+        value: "Unable to reset VPN configuration. Please try again. If the issue persists, contact support for assistance.",
         comment: "Message to show when vpn configuration reset fails.")
 
+    public static let resetVPNErrorButtonActionTitle =
+      NSLocalizedString("vpn.resetVPNErrorButtonActionTitle", tableName: "BraveShared", bundle: .module,
+        value: "Try Again",
+        comment: "Title of button to try again when vpn configuration reset fails.")
+
     public static let resetVPNSuccessTitle =
       NSLocalizedString("vpn.resetVPNSuccessTitle", tableName: "BraveShared", bundle: .module,
-        value: "Success",
+        value: "Reset Successful",
         comment: "Title for success message when vpn configuration reset succeeds.")
 
     public static let resetVPNSuccessBody =
       NSLocalizedString("vpn.resetVPNSuccessBody", tableName: "BraveShared", bundle: .module,
-        value: "VPN Configuration is resetted successfully.",
+        value: "VPN configuration has been reset successfully.",
         comment: "Message to show when vpn configuration reset succeeds.")
 
     public static let contactFormDoNotEditText =

diff --git a/Sources/BraveVPN/BraveVPNSettingsViewController.swift b/Sources/BraveVPN/BraveVPNSettingsViewController.swift
@@ -313,8 +313,17 @@ public class BraveVPNSettingsViewController: TableViewController {
       message: success ? Strings.VPN.resetVPNSuccessBody : Strings.VPN.resetVPNErrorBody,
       preferredStyle: .alert)
 
-    let okAction = UIAlertAction(title: Strings.OKString, style: .default)
-    alert.addAction(okAction)
+    if success {
+      let okAction = UIAlertAction(title: Strings.OKString, style: .default)
+      alert.addAction(okAction)
+    } else {
+      alert.addAction(UIAlertAction(title: Strings.close, style: .cancel, handler: nil))
+      alert.addAction(UIAlertAction(
+        title: Strings.VPN.resetVPNErrorButtonActionTitle, style: .default,
+        handler: { [weak self] _ in
+          self?.resetConfigurationTapped()
+      }))
+    }
 
     present(alert, animated: true)
   }