remove value:0 where it's not needed
This commit is contained in:
parent
0e77694f5e
commit
80d033aa02
|
@ -48,7 +48,7 @@ export const estimateDataGas = (
|
||||||
'',
|
'',
|
||||||
)}000000000000000000000000000000000000000000000000000000000000000001`
|
)}000000000000000000000000000000000000000000000000000000000000000001`
|
||||||
const payload = safe.contract.methods
|
const payload = safe.contract.methods
|
||||||
.execTransaction(to, valueInWei, data, operation, txGasEstimate, 0, gasPrice, gasToken, refundReceiver, sigs, { value: 0 })
|
.execTransaction(to, valueInWei, data, operation, txGasEstimate, 0, gasPrice, gasToken, refundReceiver, sigs)
|
||||||
.encodeABI()
|
.encodeABI()
|
||||||
|
|
||||||
// eslint-disable-next-line
|
// eslint-disable-next-line
|
||||||
|
@ -122,7 +122,7 @@ export const calculateTxFee = async (
|
||||||
'0x0000000000000000000000000000000000000000',
|
'0x0000000000000000000000000000000000000000',
|
||||||
'0x0000000000000000000000000000000000000000',
|
'0x0000000000000000000000000000000000000000',
|
||||||
sigs,
|
sigs,
|
||||||
{ from: '0xbc2BB26a6d821e69A38016f3858561a1D80d4182', value: 0 },
|
{ from: '0xbc2BB26a6d821e69A38016f3858561a1D80d4182' },
|
||||||
)
|
)
|
||||||
|
|
||||||
return estimate
|
return estimate
|
||||||
|
|
|
@ -33,7 +33,7 @@ export const estimateTxGasCosts = async (
|
||||||
'',
|
'',
|
||||||
)}000000000000000000000000000000000000000000000000000000000000000001`
|
)}000000000000000000000000000000000000000000000000000000000000000001`
|
||||||
txData = await safeInstance.methods
|
txData = await safeInstance.methods
|
||||||
.execTransaction(to, tx ? tx.value : 0, data, CALL, 0, 0, 0, ZERO_ADDRESS, ZERO_ADDRESS, signatures, { value: 0 })
|
.execTransaction(to, tx ? tx.value : 0, data, CALL, 0, 0, 0, ZERO_ADDRESS, ZERO_ADDRESS, signatures)
|
||||||
.encodeABI()
|
.encodeABI()
|
||||||
} else {
|
} else {
|
||||||
const txHash = await safeInstance.methods
|
const txHash = await safeInstance.methods
|
||||||
|
|
|
@ -113,7 +113,7 @@ const createTransaction = (
|
||||||
showSnackbar(notificationsQueue.afterExecutionError, enqueueSnackbar, closeSnackbar)
|
showSnackbar(notificationsQueue.afterExecutionError, enqueueSnackbar, closeSnackbar)
|
||||||
|
|
||||||
const executeDataUsedSignatures = safeInstance.contract.methods
|
const executeDataUsedSignatures = safeInstance.contract.methods
|
||||||
.execTransaction(to, valueInWei, txData, CALL, 0, 0, 0, ZERO_ADDRESS, ZERO_ADDRESS, sigs, { value: 0 })
|
.execTransaction(to, valueInWei, txData, CALL, 0, 0, 0, ZERO_ADDRESS, ZERO_ADDRESS, sigs)
|
||||||
.encodeABI()
|
.encodeABI()
|
||||||
const errMsg = await getErrorMessage(safeInstance.address, 0, executeDataUsedSignatures, from)
|
const errMsg = await getErrorMessage(safeInstance.address, 0, executeDataUsedSignatures, from)
|
||||||
console.error(`Error executing the TX: ${errMsg}`)
|
console.error(`Error executing the TX: ${errMsg}`)
|
||||||
|
|
|
@ -147,7 +147,7 @@ const processTransaction = (
|
||||||
closeSnackbar(pendingExecutionKey)
|
closeSnackbar(pendingExecutionKey)
|
||||||
showSnackbar(notificationsQueue.afterExecutionError, enqueueSnackbar, closeSnackbar)
|
showSnackbar(notificationsQueue.afterExecutionError, enqueueSnackbar, closeSnackbar)
|
||||||
|
|
||||||
const executeData = safeInstance.contract.methods.approveHash(txHash, { value: 0 }).encodeABI()
|
const executeData = safeInstance.contract.methods.approveHash(txHash).encodeABI()
|
||||||
const errMsg = await getErrorMessage(safeInstance.address, 0, executeData, from)
|
const errMsg = await getErrorMessage(safeInstance.address, 0, executeData, from)
|
||||||
console.error(`Error executing the TX: ${errMsg}`)
|
console.error(`Error executing the TX: ${errMsg}`)
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue