Page MenuHomePhabricator

[electrum] Make the coin consolidator not burn alp tokens
ClosedPublic

Authored by PiRK on Fri, Mar 14, 12:39.

Details

Summary

Change the "Include SLP tokens" option to a more generic "Inlude tokens" that covers both SLP and ALP.
Don't burn ALP tokens unless this option is checked.
Update the warning message.

Test Plan

python test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

PiRK requested review of this revision.Fri, Mar 14, 12:39

TIL coin consolidator is a thing in electrum

how is this used in the GUI? is this just called in normal tx building?

TIL coin consolidator is a thing in electrum

how is this used in the GUI? is this just called in normal tx building?

You find it in the context menu in the coins tab and in the address tab. It is a way to reduce the number of utxos in a single address, as a way to work around the issue of the automatic coin selection algorithm making transactions that are too large because is insists on spending all coins for a given address at the same time.

Screenshot from 2025-03-15 08-05-37.png (467×864 px, 133 KB)

Screenshot from 2025-03-15 08-05-55.png (467×864 px, 65 KB)

https://texplorer.bitcoinabc.org/block/00000000000f3757ae15ce982da1681ae249e4f0d8619f9b10b34e91fd8e4e97

This revision is now accepted and ready to land.Sat, Mar 15, 22:49