From 20be7cc8b7109b724c03628ccf6d06ff984ca40c Mon Sep 17 00:00:00 2001
From: Rui <102453770+ruixhuang@users.noreply.github.com>
Date: Wed, 18 Sep 2024 12:54:36 -0700
Subject: [PATCH] Sync system notification settings; filter out Abacus
 notifications (#258)

* WIP

* WIP

* Settings

* Lint

* Clean up

* Clean up

* WIP

* WIP

* Update app settings when system settings change

* Revert

* Clean up
---
 .../dydxNotificationPrimerViewPresenter.swift |  3 +++
 ...dydxNotificationsSettingsViewBuilder.swift | 13 ++++++++-
 .../dydxTradeStatusViewBuilder.swift          |  2 +-
 .../dydxStateManager/AbacusStateManager.swift |  3 +++
 .../_v4/Banner/SwiftMessages+Banner.swift     |  2 +-
 .../Primers/dydxNotificationPrimerView.swift  |  7 +++++
 .../_Localizer/dydxAbacusDataLocalizer.swift  | 27 ++++++++++++-------
 7 files changed, 44 insertions(+), 13 deletions(-)

diff --git a/dydx/dydxPresenters/dydxPresenters/_v4/Primers/dydxNotificationPrimerViewPresenter.swift b/dydx/dydxPresenters/dydxPresenters/_v4/Primers/dydxNotificationPrimerViewPresenter.swift
index 8100e3caa..1a38053ba 100644
--- a/dydx/dydxPresenters/dydxPresenters/_v4/Primers/dydxNotificationPrimerViewPresenter.swift
+++ b/dydx/dydxPresenters/dydxPresenters/_v4/Primers/dydxNotificationPrimerViewPresenter.swift
@@ -46,5 +46,8 @@ private class dydxNotificationPrimerViewPresenter: HostedViewPresenter<dydxNotif
                 }
             }
         }
+        viewModel?.cancelAction = {
+            Router.shared?.navigate(to: RoutingRequest(path: "/action/dismiss", params: nil), animated: true, completion: nil)
+        }
     }
 }
diff --git a/dydx/dydxPresenters/dydxPresenters/_v4/Profile/Notifications/dydxNotificationsSettingsViewBuilder.swift b/dydx/dydxPresenters/dydxPresenters/_v4/Profile/Notifications/dydxNotificationsSettingsViewBuilder.swift
index 710d4db13..71270aa57 100644
--- a/dydx/dydxPresenters/dydxPresenters/_v4/Profile/Notifications/dydxNotificationsSettingsViewBuilder.swift
+++ b/dydx/dydxPresenters/dydxPresenters/_v4/Profile/Notifications/dydxNotificationsSettingsViewBuilder.swift
@@ -40,6 +40,14 @@ private class dydxNotificationsSettingsViewPresenter: SettingsViewPresenter {
         viewModel?.headerViewModel = header
     }
 
+    override func start() {
+        super.start()
+
+        changeObservation(from: nil, to: NotificationService.shared, keyPath: #keyPath(NotificationHandler.permission)) { [weak self] _, _, _, _ in
+            self?.reloadSettings()
+        }
+    }
+
     override func onInputValueChanged(input: FieldInput) {
         if input.fieldName == "should_display_in_app_notifications" {
            promptToToggleNotification()
@@ -57,7 +65,10 @@ private class dydxNotificationsSettingsViewPresenter: SettingsViewPresenter {
                                                      cancelTitle: DataLocalizer.shared?.localize(path: "APP.GENERAL.CANCEL", params: nil) ?? "Cancel"
             )
         }
-        // sync the settings with the system permission
+        reloadSettings()
+    }
+
+    private func reloadSettings() {
         let pushNotificationEnabled = NotificationService.shared?.permission == .authorized
         SettingsStore.shared?.setValue(pushNotificationEnabled, forKey: dydxSettingsStoreKey.shouldDisplayInAppNotifications.rawValue)
         loadSettings()
diff --git a/dydx/dydxPresenters/dydxPresenters/_v4/Trade/TradeStatus/dydxTradeStatusViewBuilder.swift b/dydx/dydxPresenters/dydxPresenters/_v4/Trade/TradeStatus/dydxTradeStatusViewBuilder.swift
index f0264b5b3..3e7ddcd31 100644
--- a/dydx/dydxPresenters/dydxPresenters/_v4/Trade/TradeStatus/dydxTradeStatusViewBuilder.swift
+++ b/dydx/dydxPresenters/dydxPresenters/_v4/Trade/TradeStatus/dydxTradeStatusViewBuilder.swift
@@ -290,7 +290,7 @@ private extension Abacus.OrderStatus {
     var statusIcon: dydxTradeStatusLogoViewModel.StatusIcon? {
         switch self {
         case .canceled: return .failed
-        case .canceling, .pending, .partiallyfilled: return .pending
+        case .canceling, .pending, .partiallyfilled, .partiallycanceled: return .pending
         case .filled: return .filled
         case .open, .untriggered: return .open
         default: return nil
diff --git a/dydx/dydxStateManager/dydxStateManager/AbacusStateManager.swift b/dydx/dydxStateManager/dydxStateManager/AbacusStateManager.swift
index dbc2c9694..f04428c30 100644
--- a/dydx/dydxStateManager/dydxStateManager/AbacusStateManager.swift
+++ b/dydx/dydxStateManager/dydxStateManager/AbacusStateManager.swift
@@ -160,6 +160,9 @@ public final class AbacusStateManager: NSObject {
             }
         }
 
+        appConfigs.accountConfigs.subaccountConfigs.notifications = [
+            NotificationProviderType.blockreward, NotificationProviderType.positions
+        ]
         appConfigs.onboardingConfigs.alchemyApiKey = CredientialConfig.shared.credential(for: "alchemyApiKey")
         appConfigs.staticTyping = dydxBoolFeatureFlag.abacus_static_typing.isEnabled
 
diff --git a/dydx/dydxViews/dydxViews/_v4/Banner/SwiftMessages+Banner.swift b/dydx/dydxViews/dydxViews/_v4/Banner/SwiftMessages+Banner.swift
index d3a1b1f1c..b8ce88224 100644
--- a/dydx/dydxViews/dydxViews/_v4/Banner/SwiftMessages+Banner.swift
+++ b/dydx/dydxViews/dydxViews/_v4/Banner/SwiftMessages+Banner.swift
@@ -17,7 +17,7 @@ public extension MessageView {
         let iconImage: UIImage?
         switch type {
         case .info, .wait, .success:
-            backgroundColor = UIColor { _ in ThemeColor.SemanticColor.colorPurple.uiColor }
+            backgroundColor = UIColor { _ in ThemeColor.SemanticColor.layer5.uiColor }
             foregroundColor = UIColor { _ in ThemeColor.SemanticColor.colorWhite.uiColor }
             iconImage = IconStyle.default.image(theme: .info)
         case .warning:
diff --git a/dydx/dydxViews/dydxViews/_v4/Primers/dydxNotificationPrimerView.swift b/dydx/dydxViews/dydxViews/_v4/Primers/dydxNotificationPrimerView.swift
index 2f5df6c5a..bef08158e 100644
--- a/dydx/dydxViews/dydxViews/_v4/Primers/dydxNotificationPrimerView.swift
+++ b/dydx/dydxViews/dydxViews/_v4/Primers/dydxNotificationPrimerView.swift
@@ -12,6 +12,7 @@ import Utilities
 
 public class dydxNotificationPrimerViewModel: PlatformViewModel {
     @Published public var ctaAction: (() -> Void)?
+    @Published public var cancelAction: (() -> Void)?
 
     public init() { }
 
@@ -38,6 +39,12 @@ public class dydxNotificationPrimerViewModel: PlatformViewModel {
                         self?.ctaAction?()
                     }
                     .createView(parentStyle: style)
+
+                    let cancelText = Text(DataLocalizer.localize(path: "APP.GENERAL.CANCEL", params: nil))
+                    PlatformButtonViewModel(content: cancelText.wrappedViewModel, state: .secondary) { [weak self] in
+                        self?.cancelAction?()
+                    }
+                    .createView(parentStyle: style)
                 }
                     .padding([.leading, .trailing])
                     .padding(.top, 40)
diff --git a/dydxV4/dydxV4/_Localizer/dydxAbacusDataLocalizer.swift b/dydxV4/dydxV4/_Localizer/dydxAbacusDataLocalizer.swift
index ac34b3c21..affda1634 100644
--- a/dydxV4/dydxV4/_Localizer/dydxAbacusDataLocalizer.swift
+++ b/dydxV4/dydxV4/_Localizer/dydxAbacusDataLocalizer.swift
@@ -36,7 +36,10 @@ public class dydxAbacusDataLocalizer: DataLocalizerProtocol, AbacusLocalizerProt
         let language = keyValueStore?.value(forKey: _languageTag) as? String
         UIImplementations.reset(language: language)
         if let language = language {
-            setLanguage(language: language) { _ in
+            setLanguage(language: language) { success in
+                if !success {
+                    Console.shared.log("dydxAbacusDataLocalizer setLanguage failed")
+                }
             }
         }
     }
@@ -49,21 +52,25 @@ public class dydxAbacusDataLocalizer: DataLocalizerProtocol, AbacusLocalizerProt
     
     public func setLanguage(language: String, callback: @escaping (KotlinBoolean, ParsingError?) -> Void) {
         if let code = language.components(separatedBy: "-").first {
-            (UIImplementations.shared?.localizer as? DynamicLocalizer)?.setLanguage(language: code, callback: { [weak self] successful, error in
-                self?.language = (UIImplementations.shared?.localizer as? DynamicLocalizer)?.language
-                if successful.boolValue {
-                    if let self = self {
-                        self.keyValueStore?.setValue(code, forKey: self._languageTag)
+            let localizer = UIImplementations.shared?.localizer as? DynamicLocalizer
+            if let localizer = localizer {
+                localizer.setLanguage(language: code, callback: { [weak self] successful, error in
+                    self?.language = localizer.language
+                    if successful.boolValue {
+                        if let self = self {
+                            self.keyValueStore?.setValue(code, forKey: self._languageTag)
+                        }
                     }
-                }
-                callback(successful, error)
-            })
+                    callback(successful, error)
+                })
+            } else {
+                callback(false, nil)
+            }
         } else {
             callback(false, nil)
         }
     }
     
-
     private func json(params: [String: String]?) -> String? {
         if let params = params {
             if let data = try? JSONSerialization.data(withJSONObject: params, options: .prettyPrinted) {