From 1a164cd3e4709270f8123673031b9a6c61bda12f Mon Sep 17 00:00:00 2001
From: Erik Marks <25517051+rekmarks@users.noreply.github.com>
Date: Fri, 30 Oct 2020 12:22:21 -0700
Subject: [PATCH] Add confirmation for network dropdown delete action (#9763)

---
 .../app/dropdowns/network-dropdown.js           | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/ui/app/components/app/dropdowns/network-dropdown.js b/ui/app/components/app/dropdowns/network-dropdown.js
index 14c5b6635..507e9f3b3 100644
--- a/ui/app/components/app/dropdowns/network-dropdown.js
+++ b/ui/app/components/app/dropdowns/network-dropdown.js
@@ -42,9 +42,6 @@ function mapDispatchToProps (dispatch) {
     setRpcTarget: (target, chainId, ticker, nickname) => {
       dispatch(actions.setRpcTarget(target, chainId, ticker, nickname))
     },
-    delRpcTarget: (target) => {
-      dispatch(actions.delRpcTarget(target))
-    },
     hideNetworkDropdown: () => dispatch(actions.hideNetworkDropdown()),
     setNetworksTabAddMode: (isInAddMode) => {
       dispatch(actions.setNetworksTabAddMode(isInAddMode))
@@ -55,6 +52,13 @@ function mapDispatchToProps (dispatch) {
     displayInvalidCustomNetworkAlert: (networkName) => {
       dispatch(displayInvalidCustomNetworkAlert(networkName))
     },
+    showConfirmDeleteNetworkModal: ({ target, onConfirm }) => {
+      return dispatch(actions.showModal({
+        name: 'CONFIRM_DELETE_NETWORK',
+        target,
+        onConfirm,
+      }))
+    },
   }
 }
 
@@ -79,8 +83,8 @@ class NetworkDropdown extends Component {
     frequentRpcListDetail: PropTypes.array.isRequired,
     networkDropdownOpen: PropTypes.bool.isRequired,
     history: PropTypes.object.isRequired,
-    delRpcTarget: PropTypes.func.isRequired,
     displayInvalidCustomNetworkAlert: PropTypes.func.isRequired,
+    showConfirmDeleteNetworkModal: PropTypes.func.isRequired,
   }
 
   handleClick (newProviderType) {
@@ -151,7 +155,10 @@ class NetworkDropdown extends Component {
                   className="fa fa-times delete"
                   onClick={(e) => {
                     e.stopPropagation()
-                    this.props.delRpcTarget(rpcUrl)
+                    this.props.showConfirmDeleteNetworkModal({
+                      target: rpcUrl,
+                      onConfirm: () => undefined,
+                    })
                   }}
                 />
               )