Skip to content

refactor(wallet): use Amount everywhere#1595

Merged
notmandatory merged 1 commit intobitcoindevkit:masterfrom
ValuedMammal:refactor/Amount
Sep 9, 2024
Merged

refactor(wallet): use Amount everywhere#1595
notmandatory merged 1 commit intobitcoindevkit:masterfrom
ValuedMammal:refactor/Amount

Conversation

@ValuedMammal
Copy link
Copy Markdown
Collaborator

This is a followup to #1426 that refactors wallet internals to use bitcoin::Amount (nearly) everywhere. I chose not to change any public types in coin_selection.rs that still use u64 as that might require more discussion.

partially addresses #1432
fixes #1434

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

example: Fix wallet examples

3 participants