AND-9337, AND-9340, AND-9341, AND-9348, AND-9349, AND-9350, AND-9364: Fix KRC-20 issues #855
Conversation
nzeeei
force-pushed
the
fix/krc-20-issues
branch
from
7fc12bf to
4b8dcad
Compare
|
|
||
| if (change < BigDecimal.ZERO) { // unspentsToSpend not enough to cover transaction amount |
There was a problem hiding this comment.
а куда уехала эта логика с max 84 utxo?
There was a problem hiding this comment.
Проверка на utxo есть в checkUtxoAmountLimit(), логику этого метода обновил
nzeeei
force-pushed
the
fix/krc-20-issues
branch
2 times, most recently
from
a172aee to
a15cfaf
Compare
| @@ -10,7 +10,7 @@ interface UtxoAmountLimitProvider { | |||
| /** | |||
| * Checks if [amount] and [fee] is not greater than utxo limit. Returns limit otherwise. | |||
| */ | |||
| fun checkUtxoAmountLimit(amount: BigDecimal, fee: BigDecimal): UtxoAmountLimit? | |||
| fun checkUtxoAmountLimit(amount: BigDecimal, fee: BigDecimal, currencyType: CryptoCurrencyType): UtxoAmountLimit? | |||
There was a problem hiding this comment.
Эта функция для KRC20 токенов не имеет смысла.
CryptoCurrencyType - это вообще какая-то фигня, есть же AmountType, если нужно отдельный кейс только для токенов и коинов, то надо бы вложенный сделать, но никак не отдельный. Надо убрать это в принципе.
А есть какая-то причина для этого интерфейса вообще? Чем нас validateTransaction не устраивает?
There was a problem hiding this comment.
Ревертнул, в приложении сделаю так, чтобы для токенов проверки не было
| is Result.Success -> { | ||
| transactionBuilder.unspentOutputs = balanceResponse.data.unspentOutputs | ||
| } | ||
| is Result.Failure -> Unit |
There was a problem hiding this comment.
successOr лаконичнее будет
… Fix KRC-20 issues
nzeeei
force-pushed
the
fix/krc-20-issues
branch
from
a15cfaf to
0172c0d
Compare
No description provided.