Skip to content

Conversation

@Thoralf-M
Copy link
Member

Fixes #293

@Thoralf-M Thoralf-M linked an issue Oct 16, 2025 that may be closed by this pull request
Comment on lines +427 to +430
/// This method combines the balances of multiple coins of the same coin
/// type into a single coin. The `primary_coin` will receive the balances
/// from all `consumed_coins`. After merging, the `consumed_coins` will
/// be consumed and no longer exist.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could be added to the FFI crate?

Copy link
Contributor

@Alex6323 Alex6323 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll approve assuming Thibault's comment gets addressed.

@DaughterOfMars DaughterOfMars merged commit 3a336e5 into sdk-bindings Oct 23, 2025
14 checks passed
@DaughterOfMars DaughterOfMars deleted the docs/merge-coins branch October 23, 2025 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tx builder: document TransactionBuilder::merge_coins()

5 participants