From 9b01101fe3e742806ed73d26ca8c1852fe37605f Mon Sep 17 00:00:00 2001
From: mike-dydx <149746839+mike-dydx@users.noreply.github.com>
Date: Mon, 13 May 2024 16:08:47 -0400
Subject: [PATCH] remove `if DEBUG` check for sltp functionality, use abacus
 feature flag for reduce only (#163)

* remove if DEBUG checks that block new sltp UI

* reduce only feature flag driven by env.json
---
 .../_Utils/dydxFeatureFlag.swift              |  1 -
 .../dydxPresenters/_Features/features.json    | 20 -------------------
 .../dydxMarketPositionViewPresenter.swift     |  4 ----
 .../dydxTakeProfitStopLossViewPresenter.swift |  2 --
 .../dydxTradeInputEditPresenter.swift         |  2 +-
 .../dydxTakeProfitStopLossViewModel.swift     |  2 --
 6 files changed, 1 insertion(+), 30 deletions(-)

diff --git a/dydx/dydxFormatter/dydxFormatter/_Utils/dydxFeatureFlag.swift b/dydx/dydxFormatter/dydxFormatter/_Utils/dydxFeatureFlag.swift
index 3873f2798..4b16accf4 100644
--- a/dydx/dydxFormatter/dydxFormatter/_Utils/dydxFeatureFlag.swift
+++ b/dydx/dydxFormatter/dydxFormatter/_Utils/dydxFeatureFlag.swift
@@ -13,7 +13,6 @@ public enum dydxBoolFeatureFlag: String, CaseIterable {
     case push_notification
     case force_mainnet
     case enable_trading_rewards
-    case enable_reduce_only
     case enable_app_rating
     case enable_isolated_margins
 
diff --git a/dydx/dydxPresenters/dydxPresenters/_Features/features.json b/dydx/dydxPresenters/dydxPresenters/_Features/features.json
index 179715c05..018ebf311 100644
--- a/dydx/dydxPresenters/dydxPresenters/_Features/features.json
+++ b/dydx/dydxPresenters/dydxPresenters/_Features/features.json
@@ -53,26 +53,6 @@
                     ]
                 }
             },
-            {
-                "title":{
-                    "text":"Enable Reduce-only"
-                },
-                "field":{
-                    "field":"enable_reduce_only",
-                    "optional":true,
-                    "type" : "text",
-                    "options" : [
-                        {
-                            "text": "yes",
-                            "value" : "1"
-                        },
-                        {
-                            "text": "no",
-                            "value" : "0"
-                        }
-                    ]
-                }
-            },
             {
                 "title":{
                     "text":"Push Notification"
diff --git a/dydx/dydxPresenters/dydxPresenters/_v4/MarketInfo/Components/dydxMarketPositionViewPresenter.swift b/dydx/dydxPresenters/dydxPresenters/_v4/MarketInfo/Components/dydxMarketPositionViewPresenter.swift
index e4cc27f57..3354b63fd 100644
--- a/dydx/dydxPresenters/dydxPresenters/_v4/MarketInfo/Components/dydxMarketPositionViewPresenter.swift
+++ b/dydx/dydxPresenters/dydxPresenters/_v4/MarketInfo/Components/dydxMarketPositionViewPresenter.swift
@@ -78,9 +78,6 @@ class dydxMarketPositionViewPresenter: HostedViewPresenter<dydxMarketPositionVie
 
         viewModel?.funding = SignedAmountViewModel(amount: position.netFunding?.doubleValue, displayType: .dollar, coloringOption: .allText)
 
-        // hide for now until feature work complete and E2E testing
-        #if DEBUG
-        // TODO: comment this in when available
         if AbacusStateManager.shared.environment?.featureFlags.isSlTpEnabled == true {
             let routeToTakeProfitStopLossAction = {[weak self] in
                 if let marketId = self?.position?.id {
@@ -150,6 +147,5 @@ class dydxMarketPositionViewPresenter: HostedViewPresenter<dydxMarketPositionVie
 
             viewModel?.takeProfitStopLossAction = routeToTakeProfitStopLossAction
         }
-        #endif
     }
 }
diff --git a/dydx/dydxPresenters/dydxPresenters/_v4/TakeProfitStopLoss/dydxTakeProfitStopLossViewPresenter.swift b/dydx/dydxPresenters/dydxPresenters/_v4/TakeProfitStopLoss/dydxTakeProfitStopLossViewPresenter.swift
index 3c991bf5c..6e12de629 100644
--- a/dydx/dydxPresenters/dydxPresenters/_v4/TakeProfitStopLoss/dydxTakeProfitStopLossViewPresenter.swift
+++ b/dydx/dydxPresenters/dydxPresenters/_v4/TakeProfitStopLoss/dydxTakeProfitStopLossViewPresenter.swift
@@ -301,9 +301,7 @@ private class dydxTakeProfitStopLossViewPresenter: HostedViewPresenter<dydxTakeP
         viewModel.takeProfitStopLossInputAreaViewModel?.takeProfitPriceInputViewModel = .init(title: DataLocalizer.shared?.localize(path: "APP.TRIGGERS_MODAL.TP_PRICE", params: nil))
         viewModel.takeProfitStopLossInputAreaViewModel?.stopLossPriceInputViewModel = .init(title: DataLocalizer.shared?.localize(path: "APP.TRIGGERS_MODAL.SL_PRICE", params: nil))
 
-        #if DEBUG
         viewModel.shouldDisplayCustomLimitPriceViewModel = AbacusStateManager.shared.environment?.featureFlags.isSlTpLimitOrdersEnabled == true
-        #endif
 
         viewModel.customAmountViewModel = dydxCustomAmountViewModel()
 
diff --git a/dydx/dydxPresenters/dydxPresenters/_v4/Trade/TradeInput/Components/TradeInputFields/dydxTradeInputEditPresenter.swift b/dydx/dydxPresenters/dydxPresenters/_v4/Trade/TradeInput/Components/TradeInputFields/dydxTradeInputEditPresenter.swift
index dbf96b288..681a9a26e 100644
--- a/dydx/dydxPresenters/dydxPresenters/_v4/Trade/TradeInput/Components/TradeInputFields/dydxTradeInputEditPresenter.swift
+++ b/dydx/dydxPresenters/dydxPresenters/_v4/Trade/TradeInput/Components/TradeInputFields/dydxTradeInputEditPresenter.swift
@@ -230,7 +230,7 @@ internal class dydxTradeInputEditViewPresenter: HostedViewPresenter<dydxTradeInp
             visible.append(executionViewModel)
         }
 
-        if dydxBoolFeatureFlag.enable_reduce_only.isEnabled {
+        if AbacusStateManager.shared.environment?.featureFlags.reduceOnlySupported == true {
             if tradeInput.options?.needsReduceOnly == true {
                 let vm = reduceOnlyViewModel()
                 vm.isEnabled = tradeInput.options?.needsReduceOnly == true
diff --git a/dydx/dydxViews/dydxViews/_v4/TakeProfitStopLoss/dydxTakeProfitStopLossViewModel.swift b/dydx/dydxViews/dydxViews/_v4/TakeProfitStopLoss/dydxTakeProfitStopLossViewModel.swift
index 4509a1143..9d05302a8 100644
--- a/dydx/dydxViews/dydxViews/_v4/TakeProfitStopLoss/dydxTakeProfitStopLossViewModel.swift
+++ b/dydx/dydxViews/dydxViews/_v4/TakeProfitStopLoss/dydxTakeProfitStopLossViewModel.swift
@@ -147,11 +147,9 @@ public class dydxTakeProfitStopLossViewModel: PlatformViewModel {
                             self.takeProfitStopLossInputAreaViewModel?.createView(parentStyle: parentStyle, styleKey: styleKey)
                             self.separator
                             self.customAmountViewModel?.createView(parentStyle: parentStyle, styleKey: styleKey)
-                            #if DEBUG
                             if self.shouldDisplayCustomLimitPriceViewModel {
                                 self.customLimitPriceViewModel?.createView(parentStyle: parentStyle, styleKey: styleKey)
                             }
-                            #endif
                         }
                     }
                     .keyboardObserving()