Skip to content

Commit

Permalink
fix(lending): ensure supplyAsset button text appears
Browse files Browse the repository at this point in the history
- a bug was reported that asset.balance is a string so none of these conditionals pass (#245)
- cast asset.balance to a Number so the equality checks works
  • Loading branch information
0xSake authored and x48 committed Feb 23, 2021
1 parent f83e7df commit 28aa8ac
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/components/lending/supplyAsset.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -398,9 +398,9 @@ class SupplyAsset extends Component {
fullWidth
>
<Typography className={ classes.buttonText } variant={ 'h5'}>
{ asset.balance === 0 && 'No Balance to Supply' }
{ asset.balance > 0 && supplyAmount && BigNumber(supplyAmount).gt(BigNumber(asset.balance)) && 'Insufficient Balance' }
{ asset.balance > 0 && (!supplyAmount || BigNumber(supplyAmount).lte(BigNumber(asset.balance))) && 'Supply' }
{ Number(asset.balance) === 0 && 'No Balance to Supply' }
{ Number(asset.balance) > 0 && supplyAmount && BigNumber(supplyAmount).gt(BigNumber(asset.balance)) && 'Insufficient Balance' }
{ Number(asset.balance) > 0 && (!supplyAmount || BigNumber(supplyAmount).lte(BigNumber(asset.balance))) && 'Supply' }
</Typography>
</Button>
</div>
Expand Down

0 comments on commit 28aa8ac

Please sign in to comment.