Fix bug where gas param was not a string

Prevented sending transactions. Fixes #2598
7 years ago · c6713e93ad
parent fbd04a6af6
commit c6713e93ad
2 changed files with 3 additions and 3 deletions
--- a/ui/app/components/customize-gas-modal/index.js
+++ b/ui/app/components/customize-gas-modal/index.js
@ -244,7 +244,7 @@ CustomizeGasModal.prototype.render = function () {
          min: 1,
          // max: 100000,
          step: 1,
-          onChange: value => this.convertAndSetGasLimit(value),
+          onChange: value => this.convertAndSetGasLimit(String(value)),
          title: 'Gas Limit',
          copy: 'We calculate the suggested gas limit based on network success rates.',
        }),
--- a/ui/app/components/send/send-constants.js
+++ b/ui/app/components/send/send-constants.js
@ -3,8 +3,8 @@ const { conversionUtil, multiplyCurrencies } = require('../../conversion-util')

 const MIN_GAS_PRICE_HEX = (100000000).toString(16)
 const MIN_GAS_PRICE_DEC = '100000000'
-const MIN_GAS_LIMIT_HEX = (21000).toString(16)
-const MIN_GAS_LIMIT_DEC = 21000
+const MIN_GAS_LIMIT_DEC = '21000'
+const MIN_GAS_LIMIT_HEX = (parseInt(MIN_GAS_LIMIT_DEC)).toString(16)

 const MIN_GAS_PRICE_GWEI = ethUtil.addHexPrefix(conversionUtil(MIN_GAS_PRICE_HEX, {
  fromDenomination: 'WEI',