refactor: simplify filter function and remove unnecessary comments

lib/plugins/exchange/binanceus.js

@@ -15,6 +15,4 @@ const loadConfig = (account) => {
   return { ...mapped, timeout: 3000 }
 }
 
-// Documentation mentions a mandatory field 'time', not sure how ccxt handles this, probably works if we send this via options
-
 module.exports = { loadConfig, REQUIRED_CONFIG_FIELDS, CRYPTO, FIAT, ORDER_TYPE }

lib/plugins/exchange/cex.js

@@ -15,7 +15,4 @@ const loadConfig = (account) => {
   return { ...mapped, timeout: 3000 }
 }
 
-// Documentation mentions a field 'nonce', doesn't specify if it's mandatory
-// or not but we should pay attention to this when testing
-
 module.exports = { loadConfig, REQUIRED_CONFIG_FIELDS, CRYPTO, FIAT, ORDER_TYPE }

lib/plugins/exchange/ftx.js

@@ -6,7 +6,6 @@ const { ORDER_TYPES } = require('./consts')
 const ORDER_TYPE = ORDER_TYPES.MARKET
 const { BTC, BCH, ETH, LTC, USDT } = COINS
 const CRYPTO = [BTC, ETH, LTC, BCH, USDT]
-// ftx supports EUR as well but only for certain cryptos: ETH, BTC
 const FIAT = ['USD']
 const REQUIRED_CONFIG_FIELDS = ['apiKey', 'privateKey']
 

new-lamassu-admin/src/pages/Wizard/components/Wallet/ChooseExchange.js

@@ -38,8 +38,7 @@ const SAVE_ACCOUNTS = gql`
   }
 `
 
-const isConfigurable = it =>
-  R.contains(it)(['kraken', 'itbit', 'bitstamp', 'binanceus', 'cex', 'ftx'])
+const isConfigurable = it => !R.contains(it)(['mock-exchange'])
 
 const ChooseExchange = ({ data: currentData, addData }) => {
   const classes = useStyles()