Skip to content

Commit

Permalink
Fix: bulk tx tooltip wording (#4055)
Browse files Browse the repository at this point in the history
  • Loading branch information
katspaugh authored Aug 8, 2024
1 parent 6642e84 commit c3431c7
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
3 changes: 1 addition & 2 deletions cypress/e2e/pages/create_tx.pages.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,7 @@ const bulkConfirmationText = (tx) =>

const disabledBultExecuteBtnTooltip =
'Batch execution is only available for transactions that have been fully signed and are strictly sequential in Safe Account nonce'
const enabledBulkExecuteBtnTooltip =
'All transactions highlighted in light green will be included in the batch execution'
const enabledBulkExecuteBtnTooltip = 'All highlighted transactions will be included in the batch execution'

const bulkExecuteBtnStr = 'Bulk execute'

Expand Down
2 changes: 1 addition & 1 deletion src/components/transactions/BatchExecuteButton/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ const BatchExecuteButton = () => {
title={
isDisabled
? 'Batch execution is only available for transactions that have been fully signed and are strictly sequential in Safe Account nonce.'
: 'All transactions highlighted in light green will be included in the batch execution.'
: 'All highlighted transactions will be included in the batch execution.'
}
>
<span>
Expand Down
4 changes: 2 additions & 2 deletions src/components/tx-flow/flows/ExecuteBatch/ReviewBatch.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -158,8 +158,8 @@ export const ReviewBatch = ({ params }: { params: ExecuteBatchFlowProps }) => {
<Typography variant="body2">
This transaction batches a total of {params.txs.length} transactions from your queue into a single Ethereum
transaction. Please check every included transaction carefully, especially if you have rejection transactions,
and make sure you want to execute all of them. Included transactions are highlighted in green when you hover
over the execute button.
and make sure you want to execute all of them. Included transactions are highlighted when you hover over the
execute button.
</Typography>

{multiSendContract && <SendToBlock address={multisendContractAddress} title="Interact with" />}
Expand Down

0 comments on commit c3431c7

Please sign in to comment.