From e89aaba312de14dd0f17f1ab9f9e26eaa46d20ad Mon Sep 17 00:00:00 2001
From: mike-dydx <149746839+mike-dydx@users.noreply.github.com>
Date: Thu, 6 Jun 2024 16:51:48 -0400
Subject: [PATCH] do not render svgs (#185)

* do not render svgs

* remove trading rewards feature flag

* set abacus historical trading reward period properly
---
 .../_Utils/dydxFeatureFlag.swift              |  3 +--
 .../dydxPresenters/_Features/features.json    | 20 ---------------
 .../dydxRewardsHistoryViewPresenter.swift     | 22 +++++++++++++---
 .../dydxTransferSearchViewBuilder.swift       | 25 +++++++++++--------
 .../dydxStateManager/AbacusStateManager.swift |  4 +++
 .../_v4/Profile/dydxProfileView.swift         |  6 ++---
 .../Search/dydxTransferSearchItemView.swift   |  2 +-
 7 files changed, 41 insertions(+), 41 deletions(-)

diff --git a/dydx/dydxFormatter/dydxFormatter/_Utils/dydxFeatureFlag.swift b/dydx/dydxFormatter/dydxFormatter/_Utils/dydxFeatureFlag.swift
index a47aa871f..55eddaba5 100644
--- a/dydx/dydxFormatter/dydxFormatter/_Utils/dydxFeatureFlag.swift
+++ b/dydx/dydxFormatter/dydxFormatter/_Utils/dydxFeatureFlag.swift
@@ -12,7 +12,6 @@ import Utilities
 public enum dydxBoolFeatureFlag: String, CaseIterable {
     case push_notification
     case force_mainnet
-    case enable_trading_rewards
     case enable_app_rating
     case enable_isolated_margins
 
@@ -25,7 +24,7 @@ public enum dydxBoolFeatureFlag: String, CaseIterable {
         switch self {
         case .enable_app_rating:
             return Self.obj.parser.asBoolean(FeatureService.shared?.flag(feature: rawValue))?.boolValue ?? true
-        case .push_notification, .force_mainnet, .enable_trading_rewards, .enable_isolated_margins:
+        case .push_notification, .force_mainnet, .enable_isolated_margins:
             return Self.obj.parser.asBoolean(FeatureService.shared?.flag(feature: rawValue))?.boolValue ?? false
         }
     }
diff --git a/dydx/dydxPresenters/dydxPresenters/_Features/features.json b/dydx/dydxPresenters/dydxPresenters/_Features/features.json
index 018ebf311..7a13d7891 100644
--- a/dydx/dydxPresenters/dydxPresenters/_Features/features.json
+++ b/dydx/dydxPresenters/dydxPresenters/_Features/features.json
@@ -33,26 +33,6 @@
                     ]
                 }
             },
-            {
-                "title":{
-                    "text":"Enable Trading Rewards"
-                },
-                "field":{
-                    "field":"enable_trading_rewards",
-                    "optional":true,
-                    "type" : "text",
-                    "options" : [
-                        {
-                            "text": "yes",
-                            "value" : "1"
-                        },
-                        {
-                            "text": "no",
-                            "value" : "0"
-                        }
-                    ]
-                }
-            },
             {
                 "title":{
                     "text":"Push Notification"
diff --git a/dydx/dydxPresenters/dydxPresenters/_v4/Profile/TradingRewards/Components/dydxRewardsHistoryViewPresenter.swift b/dydx/dydxPresenters/dydxPresenters/_v4/Profile/TradingRewards/Components/dydxRewardsHistoryViewPresenter.swift
index 7e09b74e2..a8f128703 100644
--- a/dydx/dydxPresenters/dydxPresenters/_v4/Profile/TradingRewards/Components/dydxRewardsHistoryViewPresenter.swift
+++ b/dydx/dydxPresenters/dydxPresenters/_v4/Profile/TradingRewards/Components/dydxRewardsHistoryViewPresenter.swift
@@ -36,6 +36,17 @@ public class dydxRewardsHistoryViewPresenter: HostedViewPresenter<dydxRewardsHis
             }
         }
 
+        var abacusPeriod: Abacus.HistoricalTradingRewardsPeriod {
+            switch self {
+            case .monthly:
+                return .monthly
+            case .weekly:
+                return .weekly
+            case .daily:
+                return .daily
+            }
+        }
+
         var text: String? {
             switch self {
             case .monthly: return DataLocalizer.shared?.localize(path: "APP.GENERAL.TIME_STRINGS.MONTHLY", params: nil)
@@ -45,7 +56,8 @@ public class dydxRewardsHistoryViewPresenter: HostedViewPresenter<dydxRewardsHis
         }
     }
 
-    @Published private var selectedPeriodIndex: Int = 0
+    @Published private var selectedPeriodIndex: Int = 1
+    private var period: Period { Period.allCases[selectedPeriodIndex] }
 
     override init() {
 
@@ -65,12 +77,15 @@ public class dydxRewardsHistoryViewPresenter: HostedViewPresenter<dydxRewardsHis
     public override func start() {
         super.start()
 
+        AbacusStateManager.shared.setHistoricalTradingRewardPeriod(period: self.period.abacusPeriod)
         AbacusStateManager.shared.state.account
             .map(\.?.tradingRewards?.historical)
             .sink { [weak self] historicalRewards in
                 self?.viewModel?.onSelectionChanged = {[weak self] index in
-                    self?.selectedPeriodIndex = index
-                    self?.updateItems(from: historicalRewards)
+                    guard let self = self else { return }
+                    self.selectedPeriodIndex = index
+                    AbacusStateManager.shared.setHistoricalTradingRewardPeriod(period: self.period.abacusPeriod)
+                    self.updateItems(from: historicalRewards)
                 }
                 self?.updateItems(from: historicalRewards)
             }
@@ -78,7 +93,6 @@ public class dydxRewardsHistoryViewPresenter: HostedViewPresenter<dydxRewardsHis
     }
 
     private func updateItems(from historicalRewards: [String: [HistoricalTradingReward]]?) {
-        let period = Period.allCases[selectedPeriodIndex]
         viewModel?.items = historicalRewards?[period.historicalMapKey]?
             .map { reward in
                 let startedAt = dydxFormatter.shared.millisecondsToDate(reward.startedAtInMilliseconds, format: .MMM_d_yyyy)
diff --git a/dydx/dydxPresenters/dydxPresenters/_v4/Transfer/Search/dydxTransferSearchViewBuilder.swift b/dydx/dydxPresenters/dydxPresenters/_v4/Transfer/Search/dydxTransferSearchViewBuilder.swift
index dd387763f..e5b3def6d 100644
--- a/dydx/dydxPresenters/dydxPresenters/_v4/Transfer/Search/dydxTransferSearchViewBuilder.swift
+++ b/dydx/dydxPresenters/dydxPresenters/_v4/Transfer/Search/dydxTransferSearchViewBuilder.swift
@@ -112,16 +112,21 @@ private extension dydxTransferSearchItemViewModel {
         }
         self.isSelected = selected
         if let iconUrl = option.iconUrl, let url = URL(string: iconUrl) {
-            self.icon = PlatformIconViewModel(
-                type: .url(url: url, placeholderContent: {
-                    Text(option.localizedString?.prefix(1) ?? "")
-                        .frame(width: 32, height: 32)
-                        .themeColor(foreground: .textTertiary)
-                        .themeColor(background: .layer5)
-                        .borderAndClip(style: .circle, borderColor: .layer7, lineWidth: 1)
-                        .wrappedInAnyView()
-                }),
-                size: CGSize(width: 32, height: 32))
+            let type: PlatformIconViewModel.IconType
+            let placeholderText = Text(option.localizedString?.prefix(1) ?? "")
+                .frame(width: 32, height: 32)
+                .themeColor(foreground: .textTertiary)
+                .themeColor(background: .layer5)
+                .borderAndClip(style: .circle, borderColor: .layer7, lineWidth: 1)
+                .wrappedInAnyView()
+            // SwiftUI does not process svgs well
+            if url.absoluteString.ends(with: ".svg") {
+                self.icon = placeholderText.wrappedViewModel
+            } else {
+                self.icon = PlatformIconViewModel(
+                    type: .url(url: url, placeholderContent: { placeholderText }),
+                    size: CGSize(width: 32, height: 32))
+            }
         }
         self.onTapAction = onTapAction
     }
diff --git a/dydx/dydxStateManager/dydxStateManager/AbacusStateManager.swift b/dydx/dydxStateManager/dydxStateManager/AbacusStateManager.swift
index b47117a81..29a38b4f4 100644
--- a/dydx/dydxStateManager/dydxStateManager/AbacusStateManager.swift
+++ b/dydx/dydxStateManager/dydxStateManager/AbacusStateManager.swift
@@ -269,6 +269,10 @@ public final class AbacusStateManager: NSObject {
         asyncStateManager.historicalPnlPeriod = period
     }
 
+    public func setHistoricalTradingRewardPeriod(period: HistoricalTradingRewardsPeriod) {
+        asyncStateManager.historicalTradingRewardPeriod = period
+    }
+
     public func startTrade() {
         asyncStateManager.trade(data: nil, type: nil)
     }
diff --git a/dydx/dydxViews/dydxViews/_v4/Profile/dydxProfileView.swift b/dydx/dydxViews/dydxViews/_v4/Profile/dydxProfileView.swift
index f27f8b1f8..75d7fc5de 100644
--- a/dydx/dydxViews/dydxViews/_v4/Profile/dydxProfileView.swift
+++ b/dydx/dydxViews/dydxViews/_v4/Profile/dydxProfileView.swift
@@ -50,10 +50,8 @@ public class dydxProfileViewModel: PlatformViewModel {
                     HStack(spacing: 14) {
                         self.fees?
                             .createView(parentStyle: style)
-                        if dydxBoolFeatureFlag.enable_trading_rewards.isEnabled {
-                            self.rewards?
-                                .createView(parentStyle: style)
-                        }
+                        self.rewards?
+                            .createView(parentStyle: style)
                     }
 
                     self.history?
diff --git a/dydx/dydxViews/dydxViews/_v4/Transfer/Search/dydxTransferSearchItemView.swift b/dydx/dydxViews/dydxViews/_v4/Transfer/Search/dydxTransferSearchItemView.swift
index 4af7c65fe..e2e02dc4c 100644
--- a/dydx/dydxViews/dydxViews/_v4/Transfer/Search/dydxTransferSearchItemView.swift
+++ b/dydx/dydxViews/dydxViews/_v4/Transfer/Search/dydxTransferSearchItemView.swift
@@ -12,7 +12,7 @@ import Utilities
 
 public class dydxTransferSearchItemViewModel: PlatformViewModel {
     @Published public var onTapAction: (() -> Void)?
-    @Published public var icon: PlatformIconViewModel?
+    @Published public var icon: PlatformViewModel?
     @Published public var text: String?
     @Published public var tokenText: TokenTextViewModel?
     @Published public var isSelected: Bool = false