feat: advanced wallet settings up to spec

lib/new-config-manager.js

@@ -54,7 +54,10 @@ const getLocale = (deviceId, it) => {
 
 const getGlobalLocale = it => getLocale(null, it)
 
-const getWalletSettings = (key, it) => _.compose(fromNamespace(key), fromNamespace(namespaces.WALLETS))(it)
+const getWalletSettings = (key, it) => {
+  const result = _.compose(fromNamespace(key), fromNamespace(namespaces.WALLETS))(it)
+  return result
+}
 const getCashOut = (key, it) => _.compose(fromNamespace(key), fromNamespace(namespaces.CASH_OUT))(it)
 const getGlobalCashOut = fromNamespace(namespaces.CASH_OUT)
 const getOperatorInfo = fromNamespace(namespaces.OPERATOR_INFO)

migrations/1645010873828-add-advanced-wallet-settings.js

@@ -0,0 +1,22 @@
+const uuid = require('uuid')
+const { saveConfig, loadLatest } = require('../lib/new-settings-loader')
+
+exports.up = function (next) {
+  const newConfig = {}
+  return loadLatest()
+    .then(config => {
+      newConfig[`wallets_advanced_feeMultiplier`] = '1'
+      newConfig[`wallets_advanced_cryptoUnits`] = 'full'
+      newConfig[`wallets_advanced_allowTransactionBatching`] = false
+      newConfig[`wallets_advanced_id`] = uuid.v4()
+      return saveConfig(newConfig)
+    })
+    .then(next)
+    .catch(err => {
+      return next(err)
+    })
+}
+
+module.exports.down = function (next) {
+  next()
+}

new-lamassu-admin/src/pages/Wallet/AdvancedWallet.js

@@ -2,27 +2,28 @@ import { useQuery, useMutation } from '@apollo/react-hooks'
 import gql from 'graphql-tag'
 import { utils as coinUtils } from 'lamassu-coins'
 import * as R from 'ramda'
-import React from 'react'
+import React, { useState } from 'react'
 
-import { NamespacedTable as EditableTable } from 'src/components/editableTable'
+import { Table as EditableTable } from 'src/components/editableTable'
+import Section from 'src/components/layout/Section'
 import { fromNamespace, toNamespace, namespaces } from 'src/utils/config'
 
 import {
-  WalletSchema,
   AdvancedWalletSchema,
-  getAdvancedWalletElements
+  getAdvancedWalletElements,
+  getAdvancedWalletElementsOverrides,
+  OverridesDefaults,
+  OverridesSchema
 } from './helper'
 
 const SAVE_CONFIG = gql`
-  mutation Save($config: JSONObject, $accounts: JSONObject) {
+  mutation Save($config: JSONObject) {
     saveConfig(config: $config)
-    saveAccounts(accounts: $accounts)
   }
 `
 const GET_INFO = gql`
   query getData {
     config
-    accounts
     cryptoCurrencies {
       code
       display
@@ -31,34 +32,112 @@ const GET_INFO = gql`
 `
 
 const AdvancedWallet = () => {
+  const ADVANCED = 'advanced'
+  const CRYPTOCURRENCY_KEY = 'cryptoCurrency'
   const SCREEN_KEY = namespaces.WALLETS
   const { data } = useQuery(GET_INFO)
 
+  const [isEditingDefault, setEditingDefault] = useState(false)
+  const [isEditingOverrides, setEditingOverrides] = useState(false)
+
   const [saveConfig, { error }] = useMutation(SAVE_CONFIG, {
     refetchQueries: () => ['getData']
   })
 
-  const save = (rawConfig, accounts) => {
+  const mapConfigKeys = R.curry((fn, obj) =>
-    const config = toNamespace(SCREEN_KEY)(rawConfig)
+    R.zipObj(R.map(fn, R.keys(obj)), R.values(obj))
-    return saveConfig({ variables: { config, accounts } })
+  )
+
+  const save = rawConfig => {
+    const config = toNamespace(SCREEN_KEY)(
+      mapConfigKeys(it => `${ADVANCED}_` + it, rawConfig.wallets[0])
+    )
+
+    return saveConfig({ variables: { config } })
   }
 
-  const config = data?.config && fromNamespace(SCREEN_KEY)(data.config)
+  const saveOverrides = rawConfig => {
+    const config = toNamespace(SCREEN_KEY)(
+      mapConfigKeys(it => `${ADVANCED}_` + it, rawConfig)
+    )
+    return saveConfig({ variables: { config } })
+  }
+
+  const onEditingDefault = (it, editing) => setEditingDefault(editing)
+  const onEditingOverrides = (it, editing) => setEditingOverrides(editing)
+
   const cryptoCurrencies = data?.cryptoCurrencies ?? []
 
+  const AdvancedWalletSettings = mapConfigKeys(
+    it => R.tail(R.split('_', it)),
+    R.pickBy(
+      (val, key) => R.head(R.split('_', key)) === ADVANCED,
+      data?.config && fromNamespace(SCREEN_KEY)(data.config)
+    )
+  )
+
+  const AdvancedWalletSettingsOverrides = AdvancedWalletSettings.overrides ?? []
+
+  const overridenCryptos = R.map(R.prop(CRYPTOCURRENCY_KEY))(
+    AdvancedWalletSettingsOverrides
+  )
+  const suggestionFilter = R.filter(
+    it => !R.contains(it.code, overridenCryptos)
+  )
+  const coinSuggestions = suggestionFilter(cryptoCurrencies)
+
+  const findSuggestion = it => {
+    const coin = R.compose(R.find(R.propEq('code', it?.cryptoCurrency)))(
+      cryptoCurrencies
+    )
+    return coin ? [coin] : []
+  }
+
   return (
-    <EditableTable
+    <>
-      name="advancedWallet"
+      <Section>
-      namespaces={R.map(R.path(['code']))(cryptoCurrencies)}
+        <EditableTable
-      data={config}
+          name="wallets"
-      error={error?.message}
+          data={R.of(AdvancedWalletSettings)}
-      enableEdit
+          error={error?.message}
-      editWidth={174}
+          enableEdit
-      save={save}
+          editWidth={174}
-      stripeWhen={it => !WalletSchema.isValidSync(it)}
+          save={save}
-      validationSchema={AdvancedWalletSchema}
+          stripeWhen={it => !AdvancedWalletSchema.isValidSync(it)}
-      elements={getAdvancedWalletElements(cryptoCurrencies, coinUtils, config)}
+          inialValues={R.of(AdvancedWalletSettings)}
-    />
+          validationSchema={AdvancedWalletSchema}
+          elements={getAdvancedWalletElements(
+            coinUtils,
+            AdvancedWalletSettings
+          )}
+          setEditing={onEditingDefault}
+          forceDisable={isEditingOverrides}
+        />
+      </Section>
+      <Section>
+        <EditableTable
+          error={error?.message}
+          title="Overrides"
+          titleLg
+          name="overrides"
+          enableDelete
+          enableEdit
+          enableCreate
+          inialValues={OverridesDefaults}
+          save={saveOverrides}
+          validationSchema={OverridesSchema}
+          data={AdvancedWalletSettingsOverrides ?? []}
+          elements={getAdvancedWalletElementsOverrides(
+            coinSuggestions,
+            findSuggestion,
+            coinUtils
+          )}
+          disableAdd={!coinSuggestions?.length}
+          setEditing={onEditingOverrides}
+          forceDisable={isEditingDefault}
+        />
+      </Section>
+    </>
   )
 }
 

new-lamassu-admin/src/pages/Wallet/Wallet.js

@@ -1,18 +1,14 @@
 import { useQuery, useMutation } from '@apollo/react-hooks'
-import { DialogActions, makeStyles, Box } from '@material-ui/core'
+import { makeStyles } from '@material-ui/core'
 import gql from 'graphql-tag'
 import * as R from 'ramda'
 import React, { useState } from 'react'
 
 import Modal from 'src/components/Modal'
-import { IconButton, Button } from 'src/components/buttons'
 import { NamespacedTable as EditableTable } from 'src/components/editableTable'
-import { RadioGroup } from 'src/components/inputs'
 import TitleSection from 'src/components/layout/TitleSection'
-import { P, Label1 } from 'src/components/typography'
 import FormRenderer from 'src/pages/Services/FormRenderer'
 import schemas from 'src/pages/Services/schemas'
-import { ReactComponent as EditIcon } from 'src/styling/icons/action/edit/enabled.svg'
 import { ReactComponent as ReverseSettingsIcon } from 'src/styling/icons/circle buttons/settings/white.svg'
 import { ReactComponent as SettingsIcon } from 'src/styling/icons/circle buttons/settings/zodiac.svg'
 import { fromNamespace, toNamespace } from 'src/utils/config'
@@ -59,8 +55,6 @@ const useStyles = makeStyles(styles)
 
 const Wallet = ({ name: SCREEN_KEY }) => {
   const classes = useStyles()
-  const [editingFeeDiscount, setEditingFeeDiscount] = useState(null)
-  const [selectedDiscount, setSelectedDiscount] = useState(null)
   const [editingSchema, setEditingSchema] = useState(null)
   const [onChangeFunction, setOnChangeFunction] = useState(null)
   const [wizard, setWizard] = useState(false)
@@ -116,25 +110,6 @@ const Wallet = ({ name: SCREEN_KEY }) => {
       return it
     })
 
-  const saveFeeDiscount = rawConfig => {
-    const config = toNamespace(SCREEN_KEY)(rawConfig)
-    setEditingFeeDiscount(false)
-    return saveConfig({ variables: { config } })
-  }
-
-  const handleRadioButtons = evt => {
-    const selectedDiscount = R.path(['target', 'value'])(evt)
-    setSelectedDiscount(selectedDiscount)
-  }
-
-  const radioButtonOptions = [
-    { display: '+20%', code: '1.2' },
-    { display: 'Default', code: '1' },
-    { display: '-20%', code: '0.8' },
-    { display: '-40%', code: '0.6' },
-    { display: '-60%', code: '0.4' }
-  ]
-
   return (
     <>
       <div className={classes.header}>
@@ -149,19 +124,6 @@ const Wallet = ({ name: SCREEN_KEY }) => {
             }
           ]}
         />
-        <Box alignItems="center" justifyContent="end">
-          <Label1 className={classes.feeDiscountLabel}>Fee discount</Label1>
-          <Box
-            display="flex"
-            alignItems="center"
-            justifyContent="end"
-            mr="-4px">
-            <P className={classes.selection}>{selectedDiscount}</P>
-            <IconButton onClick={() => setEditingFeeDiscount(true)}>
-              <EditIcon />
-            </IconButton>
-          </Box>
-        </Box>
       </div>
       {!advancedSettings && (
         <>
@@ -209,32 +171,6 @@ const Wallet = ({ name: SCREEN_KEY }) => {
         </>
       )}
       {advancedSettings && <AdvancedWallet></AdvancedWallet>}
-      {editingFeeDiscount && (
-        <Modal
-          title={'Fee discount for BTC'}
-          width={478}
-          handleClose={() => setEditingFeeDiscount(null)}
-          open={true}>
-          <P>
-            Set a priority level for your outgoing BTC transactions, selecting a
-            percentage off of the fee estimate your wallet uses.
-          </P>
-          <RadioGroup
-            name="set-automatic-reset"
-            value={selectedDiscount}
-            options={radioButtonOptions}
-            onChange={handleRadioButtons}
-          />
-          <DialogActions>
-            <Button
-              onClick={() =>
-                saveFeeDiscount({ BTC_feeMultiplier: selectedDiscount })
-              }>
-              Confirm
-            </Button>
-          </DialogActions>
-        </Modal>
-      )}
     </>
   )
 }

new-lamassu-admin/src/pages/Wallet/Wallet.styles.js

@@ -1,16 +1,7 @@
-import { offColor } from 'src/styling/variables'
-
 export default {
   header: {
     display: 'flex',
     alignItems: 'center',
     justifyContent: 'space-between'
-  },
-  feeDiscountLabel: {
-    color: offColor,
-    margin: [[13, 0, -5, 20]]
-  },
-  selection: {
-    marginRight: 12
   }
 }

new-lamassu-admin/src/pages/Wallet/helper.js

@@ -33,32 +33,50 @@ const WalletSchema = Yup.object().shape({
 
 const AdvancedWalletSchema = Yup.object().shape({
   cryptoUnits: Yup.string().required(),
+  feeMultiplier: Yup.string().required(),
   allowTransactionBatching: Yup.boolean()
 })
 
-const getAdvancedWalletElements = (cryptoCurrencies, coinUtils, config) => {
+const OverridesSchema = Yup.object().shape({
-  const viewCryptoCurrency = it =>
+  cryptoUnits: Yup.string().required(),
-    R.compose(
+  feeMultiplier: Yup.string().required(),
-      R.prop(['display']),
+  cryptoCurrency: Yup.string().required(),
-      R.find(R.propEq('code', it))
+  allowTransactionBatching: Yup.boolean()
-    )(cryptoCurrencies)
+})
 
-  const getOptions = R.curry((coinUtils, it) => {
+const OverridesDefaults = {
-    const options = R.keys(coinUtils.getCryptoCurrency(it.id).units)
+  cryptoUnits: '',
-    return R.map(option => {
+  feeMultiplier: '',
-      return { code: option, display: option }
+  cryptoCurrency: '',
-    })(options)
+  allowTransactionBatching: null
-  })
+}
 
+const viewFeeMultiplier = it =>
+  R.compose(R.prop(['display']), R.find(R.propEq('code', it)))(feeOptions)
+
+const feeOptions = [
+  { display: '+20%', code: '1.2' },
+  { display: 'Default', code: '1' },
+  { display: '-20%', code: '0.8' },
+  { display: '-40%', code: '0.6' },
+  { display: '-60%', code: '0.4' }
+]
+
+const cryptoUnitsDefaultOptions = [
+  { display: 'mili', code: 'mili' },
+  { display: 'full', code: 'full' }
+]
+
+const getCryptoUnitsOptions = R.curry((coinUtils, it) => {
+  if (R.isNil(it.cryptoCurrency)) return cryptoUnitsDefaultOptions
+  const options = R.keys(coinUtils.getCryptoCurrency(it.cryptoCurrency).units)
+  return R.map(option => {
+    return { code: option, display: option }
+  })(options)
+})
+
+const getAdvancedWalletElements = () => {
   return [
-    {
-      name: 'id',
-      header: 'Cryptocurrency',
-      width: 180,
-      view: viewCryptoCurrency,
-      size: 'sm',
-      editable: false
-    },
     {
       name: 'cryptoUnits',
       size: 'sm',
@@ -66,7 +84,7 @@ const getAdvancedWalletElements = (cryptoCurrencies, coinUtils, config) => {
       width: 190,
       input: Autocomplete,
       inputProps: {
-        options: getOptions(coinUtils),
+        options: cryptoUnitsDefaultOptions,
         valueProp: 'code',
         labelProp: 'display'
       }
@@ -77,12 +95,83 @@ const getAdvancedWalletElements = (cryptoCurrencies, coinUtils, config) => {
       stripe: true,
       width: 250,
       view: (_, ite) => {
-        if (ite.id !== 'BTC')
+        return ite.allowTransactionBatching ? 'Yes (Only BTC supports)' : `No`
+      },
+      input: Checkbox
+    },
+    {
+      name: 'feeMultiplier',
+      header: `Miner's Fee`,
+      size: 'sm',
+      stripe: true,
+      width: 250,
+      view: viewFeeMultiplier,
+      input: Autocomplete,
+      inputProps: {
+        options: feeOptions,
+        valueProp: 'code',
+        labelProp: 'display'
+      }
+    }
+  ]
+}
+
+const getAdvancedWalletElementsOverrides = (
+  coinSuggestions,
+  findSuggestion,
+  coinUtils
+) => {
+  return [
+    {
+      name: 'cryptoCurrency',
+      width: 180,
+      input: Autocomplete,
+      inputProps: {
+        options: it => R.concat(coinSuggestions, findSuggestion(it)),
+        optionsLimit: null,
+        valueProp: 'code',
+        labelProp: 'display'
+      },
+      size: 'sm'
+    },
+    {
+      name: 'cryptoUnits',
+      size: 'sm',
+      stripe: true,
+      width: 190,
+      input: Autocomplete,
+      inputProps: {
+        options: getCryptoUnitsOptions(coinUtils),
+        valueProp: 'code',
+        labelProp: 'display'
+      }
+    },
+    {
+      name: 'allowTransactionBatching',
+      size: 'sm',
+      stripe: true,
+      width: 250,
+      view: (_, ite) => {
+        if (ite.cryptoCurrency !== 'BTC')
           return <span style={classes.editDisabled}>{`No`}</span>
-        return config[`${ite.id}_allowTransactionBatching`] ? 'Yes' : 'No'
+        return ite.allowTransactionBatching ? 'Yes' : 'No'
       },
       input: Checkbox,
-      editable: it => it.id === 'BTC'
+      editable: it => it.cryptoCurrency === 'BTC'
+    },
+    {
+      name: 'feeMultiplier',
+      header: `Miner's Fee`,
+      size: 'sm',
+      stripe: true,
+      width: 250,
+      view: viewFeeMultiplier,
+      input: Autocomplete,
+      inputProps: {
+        options: feeOptions,
+        valueProp: 'code',
+        labelProp: 'display'
+      }
     }
   ]
 }
@@ -199,5 +288,8 @@ export {
   AdvancedWalletSchema,
   getElements,
   filterClass,
-  getAdvancedWalletElements
+  getAdvancedWalletElements,
+  getAdvancedWalletElementsOverrides,
+  OverridesDefaults,
+  OverridesSchema
 }