diff --git a/package.json b/package.json index eb592303..af48d2d0 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "safe-react", - "version": "2.18.1", + "version": "2.19.1", "description": "Allowing crypto users manage funds in a safer way", "website": "https://github.com/gnosis/safe-react#readme", "bugs": { diff --git a/src/components/TransactionsFees/index.tsx b/src/components/TransactionsFees/index.tsx index c5e54f19..09187210 100644 --- a/src/components/TransactionsFees/index.tsx +++ b/src/components/TransactionsFees/index.tsx @@ -21,6 +21,9 @@ export const TransactionFees = ({ txEstimationExecutionStatus, }: TransactionFailTextProps): React.ReactElement | null => { let transactionAction + if (txEstimationExecutionStatus === EstimationStatus.LOADING) { + return null + } if (isCreation) { transactionAction = 'create' } else if (isExecution) { diff --git a/src/index.tsx b/src/index.tsx index 0e1f8204..64a80101 100644 --- a/src/index.tsx +++ b/src/index.tsx @@ -26,7 +26,7 @@ Sentry.init({ dsn: SENTRY_DSN, release: `safe-react@${process.env.REACT_APP_APP_VERSION}`, integrations: [new Integrations.BrowserTracing()], - sampleRate: 0.2, + sampleRate: 0.01, }) const root = document.getElementById('root') diff --git a/src/logic/safe/transactions/gas.ts b/src/logic/safe/transactions/gas.ts index bcee4537..c7e54e0b 100644 --- a/src/logic/safe/transactions/gas.ts +++ b/src/logic/safe/transactions/gas.ts @@ -182,15 +182,17 @@ const calculateMinimumGasForTransaction = async ( const amountOfGasToTryTx = txGasEstimation + dataGasEstimation + additionalGas console.info(`Estimating transaction creation with gas amount: ${amountOfGasToTryTx}`) try { - await getGasEstimationTxResponse({ + const estimation = await getGasEstimationTxResponse({ to: safeAddress, from: safeAddress, data: estimateData, gasPrice: 0, gas: amountOfGasToTryTx, }) - console.info(`Gas estimation successfully finished with gas amount: ${amountOfGasToTryTx}`) - return amountOfGasToTryTx + if (estimation > 0) { + console.info(`Gas estimation successfully finished with gas amount: ${amountOfGasToTryTx}`) + return amountOfGasToTryTx + } } catch (error) { console.log(`Error trying to estimate gas with amount: ${amountOfGasToTryTx}`) } diff --git a/src/routes/safe/components/Settings/ManageOwners/AddOwnerModal/index.tsx b/src/routes/safe/components/Settings/ManageOwners/AddOwnerModal/index.tsx index c3121921..37a3427c 100644 --- a/src/routes/safe/components/Settings/ManageOwners/AddOwnerModal/index.tsx +++ b/src/routes/safe/components/Settings/ManageOwners/AddOwnerModal/index.tsx @@ -21,7 +21,6 @@ import ThresholdForm from './screens/ThresholdForm' const styles = createStyles({ biggerModalWindow: { width: '775px', - minHeight: '500px', height: 'auto', }, }) diff --git a/src/routes/safe/components/Settings/ManageOwners/RemoveOwnerModal/index.tsx b/src/routes/safe/components/Settings/ManageOwners/RemoveOwnerModal/index.tsx index 575046d3..6a5e8f86 100644 --- a/src/routes/safe/components/Settings/ManageOwners/RemoveOwnerModal/index.tsx +++ b/src/routes/safe/components/Settings/ManageOwners/RemoveOwnerModal/index.tsx @@ -18,7 +18,6 @@ import { TxParameters } from 'src/routes/safe/container/hooks/useTransactionPara const styles = createStyles({ biggerModalWindow: { width: '775px', - minHeight: '500px', height: 'auto', }, }) diff --git a/src/routes/safe/components/Settings/ManageOwners/RemoveOwnerModal/screens/Review/index.tsx b/src/routes/safe/components/Settings/ManageOwners/RemoveOwnerModal/screens/Review/index.tsx index ece059c9..36ec7e79 100644 --- a/src/routes/safe/components/Settings/ManageOwners/RemoveOwnerModal/screens/Review/index.tsx +++ b/src/routes/safe/components/Settings/ManageOwners/RemoveOwnerModal/screens/Review/index.tsx @@ -216,16 +216,18 @@ export const ReviewRemoveOwnerModal = ({ isTransactionExecution={isExecution} /> - - - - + {txEstimationExecutionStatus === EstimationStatus.LOADING ? null : ( + + + + + )}