1
0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-11-30 16:18:07 +01:00
metamask-extension/ui/app/pages/send/send-footer/send-footer.component.js
Mark Stacey e20ca4668e
Return Promise from addToAddressBook thunk (#8450)
`addToAddressBook` returned a thunk that didn't return a Promise,
despite doing async work. It now returns a Promise.

The callers of this action creator were updated to `await` the
completion of the operation. It was called just before redirecting the
user to a different page or closing a modal, and it seemed appropriate
to wait before doing those things.
2020-04-29 01:59:49 -03:00

143 lines
3.7 KiB
JavaScript

import React, { Component } from 'react'
import PropTypes from 'prop-types'
import PageContainerFooter from '../../../components/ui/page-container/page-container-footer'
import { CONFIRM_TRANSACTION_ROUTE, DEFAULT_ROUTE } from '../../../helpers/constants/routes'
export default class SendFooter extends Component {
static propTypes = {
addToAddressBookIfNew: PropTypes.func,
amount: PropTypes.string,
data: PropTypes.string,
clearSend: PropTypes.func,
editingTransactionId: PropTypes.string,
from: PropTypes.object,
gasLimit: PropTypes.string,
gasPrice: PropTypes.string,
gasTotal: PropTypes.string,
history: PropTypes.object,
inError: PropTypes.bool,
selectedToken: PropTypes.object,
sign: PropTypes.func,
to: PropTypes.string,
toAccounts: PropTypes.array,
tokenBalance: PropTypes.string,
unapprovedTxs: PropTypes.object,
update: PropTypes.func,
sendErrors: PropTypes.object,
gasEstimateType: PropTypes.string,
gasIsLoading: PropTypes.bool,
}
static contextTypes = {
t: PropTypes.func,
metricsEvent: PropTypes.func,
}
onCancel () {
this.props.clearSend()
this.props.history.push(DEFAULT_ROUTE)
}
async onSubmit (event) {
event.preventDefault()
const {
addToAddressBookIfNew,
amount,
data,
editingTransactionId,
from: { address: from },
gasLimit: gas,
gasPrice,
selectedToken,
sign,
to,
unapprovedTxs,
// updateTx,
update,
toAccounts,
history,
gasEstimateType,
} = this.props
const { metricsEvent } = this.context
// Should not be needed because submit should be disabled if there are errors.
// const noErrors = !amountError && toError === null
// if (!noErrors) {
// return
// }
// TODO: add nickname functionality
await addToAddressBookIfNew(to, toAccounts)
const promise = editingTransactionId
? update({
amount,
data,
editingTransactionId,
from,
gas,
gasPrice,
selectedToken,
to,
unapprovedTxs,
})
: sign({ data, selectedToken, to, amount, from, gas, gasPrice })
Promise.resolve(promise)
.then(() => {
metricsEvent({
eventOpts: {
category: 'Transactions',
action: 'Edit Screen',
name: 'Complete',
},
customVariables: {
gasChanged: gasEstimateType,
},
})
history.push(CONFIRM_TRANSACTION_ROUTE)
})
}
formShouldBeDisabled () {
const { data, inError, selectedToken, tokenBalance, gasTotal, to, gasLimit, gasIsLoading } = this.props
const missingTokenBalance = selectedToken && !tokenBalance
const gasLimitTooLow = gasLimit < 5208 // 5208 is hex value of 21000, minimum gas limit
const shouldBeDisabled = inError || !gasTotal || missingTokenBalance || !(data || to) || gasLimitTooLow || gasIsLoading
return shouldBeDisabled
}
componentDidUpdate (prevProps) {
const { inError, sendErrors } = this.props
const { metricsEvent } = this.context
if (!prevProps.inError && inError) {
const errorField = Object.keys(sendErrors).find((key) => sendErrors[key])
const errorMessage = sendErrors[errorField]
metricsEvent({
eventOpts: {
category: 'Transactions',
action: 'Edit Screen',
name: 'Error',
},
customVariables: {
errorField,
errorMessage,
},
})
}
}
render () {
return (
<PageContainerFooter
onCancel={() => this.onCancel()}
onSubmit={(e) => this.onSubmit(e)}
disabled={this.formShouldBeDisabled()}
/>
)
}
}