Skip to content

Commit

Permalink
Pass include currencyCode in spendInfo to getMaxSpendable
Browse files Browse the repository at this point in the history
  • Loading branch information
peachbits committed Jan 18, 2024
1 parent f7867f3 commit e2673f8
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 7 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

## Unreleased

- changed: Include `currencyCode` in `spendInfo` to `getMaxSpendable`

## 2.0.1 (2024-01-08)

- added: Missing asset action types (claim, claimOrder, swapNetworkFee, and transferNetworkFee).
Expand Down
18 changes: 11 additions & 7 deletions src/core/currency/wallet/currency-wallet-api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -388,20 +388,24 @@ export function makeCurrencyWalletApi(
},
async getMaxSpendable(spendInfo: EdgeSpendInfo): Promise<string> {
spendInfo = upgradeMemos(spendInfo, plugin.currencyInfo)
// Figure out which asset this is:
const upgradedCurrency = upgradeCurrencyCode({
allTokens: input.props.state.accounts[accountId].allTokens[pluginId],
currencyInfo: plugin.currencyInfo,
tokenId: spendInfo.tokenId
})

if (typeof engine.getMaxSpendable === 'function') {
// Only provide wallet info if currency requires it:
const privateKeys = unsafeMakeSpend ? walletInfo.keys : undefined

return await engine.getMaxSpendable(spendInfo, { privateKeys })
return await engine.getMaxSpendable(
{ ...spendInfo, ...upgradedCurrency },
{ privateKeys }
)
}

// Figure out which asset this is:
const { networkFeeOption, customNetworkFee } = spendInfo
const upgradedCurrency = upgradeCurrencyCode({
allTokens: input.props.state.accounts[accountId].allTokens[pluginId],
currencyInfo: plugin.currencyInfo,
tokenId: spendInfo.tokenId
})
const balance = engine.getBalance(upgradedCurrency)

// Copy all the spend targets, setting the amounts to 0
Expand Down

0 comments on commit e2673f8

Please sign in to comment.