Skip to content

Commit

Permalink
SignerPanel: Add warning for some Ethereum providers marking second+ …
Browse files Browse the repository at this point in the history
…transactions as potentially failing (#1267)
  • Loading branch information
sohkai authored Jan 23, 2020
1 parent eca03ff commit d4d7a45
Showing 1 changed file with 18 additions and 4 deletions.
22 changes: 18 additions & 4 deletions src/components/SignerPanel/ActionPathsContent.js
Original file line number Diff line number Diff line change
Expand Up @@ -284,14 +284,28 @@ class ActionPathsContent extends React.Component {
{pretransaction && (
<Info
title="Two transactions required"
mode="warning"
css={`
margin-top: ${3 * GU}px;
`}
>
This action requires two transactions to be signed in{' '}
{getProviderString('your Ethereum provider', walletProviderId)}.{' '}
{approveTransactionMessage}
Please confirm them one after another.
<p>
This action requires two transactions to be signed in{' '}
{getProviderString('your Ethereum provider', walletProviderId)}.{' '}
{approveTransactionMessage}
Please confirm them one after another.
</p>
<p
css={`
margin-top: ${1 * GU}px;
`}
>
In some situations,{' '}
{getProviderString('your Ethereum provider', walletProviderId)}{' '}
may warn you that the second transaction will fail.{' '}
<strong css="font-weight: 800">Please ignore this warning</strong>
.
</p>
</Info>
)}
<SignerButton onClick={this.handleSign} disabled={!signingEnabled}>
Expand Down

0 comments on commit d4d7a45

Please sign in to comment.