PR 5575 review comment fixes

2 files changed, 25 insertions, 24 deletions

diff --git a/app/scripts/lib/buy-eth-url.js b/app/scripts/lib/buy-eth-url.js
index 4b1572959..d434473ee 100644
--- a/app/scripts/lib/buy-eth-url.js
+++ b/app/scripts/lib/buy-eth-url.js
@@ -11,28 +11,29 @@ module.exports = getBuyEthUrl
  * network does not match any of the specified cases, or if no network is given, returns undefined.
  *
  */
-function getBuyEthUrl ({ network, amount, address }) {
-  let url
-  switch (network) {
-    case '1':
-      url = `https://buy.coinbase.com/?code=9ec56d01-7e81-5017-930c-513daa27bb6a&amount=${amount}&address=${address}&crypto_currency=ETH`
-      break
-
-    case '3':
-      url = 'https://faucet.metamask.io/'
-      break
-
-    case '4':
-      url = 'https://www.rinkeby.io/'
-      break
-
-    case '42':
-      url = 'https://github.com/kovan-testnet/faucet'
-      break
-
-    case '5':
-      url = `https://metamask.coinswitch.co/?address=${address}&to=eth`
-      break
+function getBuyEthUrl({ network, amount, address, service }) {
+	let url
+	if (typeof network !== 'undefined') {
+  	switch (network) {
+  		case '1':
+  			url = `https://buy.coinbase.com/?code=9ec56d01-7e81-5017-930c-513daa27bb6a&amount=${amount}&address=${address}&crypto_currency=ETH`
+  			break
+  		case '3':
+  			url = 'https://faucet.metamask.io/'
+  			break
+  		case '4':
+  			url = 'https://www.rinkeby.io/'
+  			break
+  		case '42':
+  			url = 'https://github.com/kovan-testnet/faucet'
+  			break
+  	}
+  } else {
+  	switch (service) {
+  		case 'coinswitch':
+  			url = `https://metamask.coinswitch.co/?address=${address}&to=eth`
+  			break
+  	}
   }
-  return url
+	return url
 }
diff --git a/ui/app/components/modals/deposit-ether-modal.js b/ui/app/components/modals/deposit-ether-modal.js
index 6686eaefa..3f74dc065 100644
--- a/ui/app/components/modals/deposit-ether-modal.js
+++ b/ui/app/components/modals/deposit-ether-modal.js
@@ -32,7 +32,7 @@ function mapDispatchToProps (dispatch) {
       dispatch(actions.buyEth({ network: '1', address, amount: 0 }))
     },
     toCoinSwitch: (address) => {
-      dispatch(actions.buyEth({ network: '5', address, amount: 0 }))
+      dispatch(actions.buyEth({ service: 'coinswitch', address, amount: 0 }))
     },
     hideModal: () => {
       dispatch(actions.hideModal())