[Customer Center] Open App Store when the user wants to update their …

…app (#4199)
RevenueCat · Oct 1, 2024 · 8835a04 · 8835a04
1 parent f073ddb
commit 8835a04
Show file tree

Hide file tree

Showing 6 changed files with 31 additions and 8 deletions.
diff --git a/RevenueCatUI/CustomerCenter/Data/CustomerCenterConfigTestData.swift b/RevenueCatUI/CustomerCenter/Data/CustomerCenterConfigTestData.swift
@@ -103,7 +103,8 @@ enum CustomerCenterConfigTestData {
                 ]
             ),
             support: .init(email: "test-support@revenuecat.com"),
-            lastPublishedAppVersion: lastPublishedAppVersion
+            lastPublishedAppVersion: lastPublishedAppVersion,
+            productId: 1
         )
     }
 

diff --git a/RevenueCatUI/CustomerCenter/Views/AppUpdateWarningView.swift b/RevenueCatUI/CustomerCenter/Views/AppUpdateWarningView.swift
@@ -26,6 +26,21 @@ struct AppUpdateWarningView: View {
     let onUpdateAppClick: () -> Void
     let onContinueAnywayClick: () -> Void
 
+    init(onUpdateAppClick: @escaping () -> Void, onContinueAnywayClick: @escaping () -> Void) {
+        self.onUpdateAppClick = onUpdateAppClick
+        self.onContinueAnywayClick = onContinueAnywayClick
+    }
+
+    init(productId: UInt, onContinueAnywayClick: @escaping () -> Void) {
+        self.init(
+            onUpdateAppClick: {
+                // productId is a positive integer, so it should be safe to construct a URL from it.
+                UIApplication.shared.open(URL(string: "https://itunes.apple.com/app/id\(productId)")!)
+            },
+            onContinueAnywayClick: onContinueAnywayClick
+        )
+    }
+
     @Environment(\.localization)
     private var localization: CustomerCenterConfigData.Localization
     @Environment(\.appearance)

diff --git a/RevenueCatUI/CustomerCenter/Views/CustomerCenterView.swift b/RevenueCatUI/CustomerCenter/Views/CustomerCenterView.swift
@@ -94,18 +94,18 @@ private extension CustomerCenterView {
         if viewModel.hasSubscriptions {
             if viewModel.subscriptionsAreFromApple,
                let screen = configuration.screens[.management] {
-                if ignoreAppUpdateWarning || viewModel.appIsLatestVersion {
-                    ManageSubscriptionsView(screen: screen,
-                                            customerCenterActionHandler: viewModel.customerCenterActionHandler)
-                } else {
+                if let productId = configuration.productId, !ignoreAppUpdateWarning && !viewModel.appIsLatestVersion {
                     AppUpdateWarningView(
-                        onUpdateAppClick: { viewModel.onAppUpdateClick() },
+                        productId: productId,
                         onContinueAnywayClick: {
                             withAnimation {
                                 ignoreAppUpdateWarning = true
                             }
                         }
                     )
+                } else {
+                    ManageSubscriptionsView(screen: screen,
+                                            customerCenterActionHandler: viewModel.customerCenterActionHandler)
                 }
             } else {
                 WrongPlatformView()

diff --git a/Sources/CustomerCenter/CustomerCenterConfigData.swift b/Sources/CustomerCenter/CustomerCenterConfigData.swift
@@ -27,17 +27,20 @@ public struct CustomerCenterConfigData {
     public let localization: Localization
     public let support: Support
     public let lastPublishedAppVersion: String?
+    public let productId: UInt?
 
     public init(screens: [Screen.ScreenType: Screen],
                 appearance: Appearance,
                 localization: Localization,
                 support: Support,
-                lastPublishedAppVersion: String?) {
+                lastPublishedAppVersion: String?,
+                productId: UInt?) {
         self.screens = screens
         self.appearance = appearance
         self.localization = localization
         self.support = support
         self.lastPublishedAppVersion = lastPublishedAppVersion
+        self.productId = productId
     }
 
     public struct Localization {
@@ -345,6 +348,7 @@ extension CustomerCenterConfigData {
         })
         self.support = Support(from: response.customerCenter.support)
         self.lastPublishedAppVersion = response.lastPublishedAppVersion
+        self.productId = response.itunesTrackId
     }
 
 }

diff --git a/Sources/Networking/Responses/CustomerCenterConfigResponse.swift b/Sources/Networking/Responses/CustomerCenterConfigResponse.swift
@@ -21,6 +21,7 @@ struct CustomerCenterConfigResponse {
 
     let customerCenter: CustomerCenter
     let lastPublishedAppVersion: String?
+    let itunesTrackId: UInt?
 
     struct CustomerCenter {
 

diff --git a/Tests/UnitTests/CustomerCenter/CustomerCenterConfigDataTests.swift b/Tests/UnitTests/CustomerCenter/CustomerCenterConfigDataTests.swift
@@ -83,7 +83,8 @@ class CustomerCenterConfigDataTests: TestCase {
                 localization: .init(locale: "en_US", localizedStrings: ["key": "value"]),
                 support: .init(email: "support@example.com")
             ),
-            lastPublishedAppVersion: "1.2.3"
+            lastPublishedAppVersion: "1.2.3",
+            itunesTrackId: 123
         )
 
         let configData = CustomerCenterConfigData(from: mockResponse)
@@ -139,6 +140,7 @@ class CustomerCenterConfigDataTests: TestCase {
         }
 
         expect(configData.lastPublishedAppVersion) == "1.2.3"
+        expect(configData.productId) == 123
     }
 
 }