Skip to content

chore: move balance type and anchors mod to chain_data#1283

Closed
startup-dreamer wants to merge 1 commit intobitcoindevkit:masterfrom
startup-dreamer:move_anchors_mod_and_balance_to_chain_data
Closed

chore: move balance type and anchors mod to chain_data#1283
startup-dreamer wants to merge 1 commit intobitcoindevkit:masterfrom
startup-dreamer:move_anchors_mod_and_balance_to_chain_data

Conversation

@startup-dreamer
Copy link
Copy Markdown

Description

Tries to Fix: bitcoindevkit/bdk_wallet#128

Moved anchore mod (dropped tx_data_traits.rs) and Balance type to chain_data.rs as suggested in This comment

All Submissions:

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

Comment thread crates/chain/src/chain_data.rs Outdated
@startup-dreamer startup-dreamer force-pushed the move_anchors_mod_and_balance_to_chain_data branch from 2872be5 to 6dea9a1 Compare February 16, 2024 21:30
@startup-dreamer
Copy link
Copy Markdown
Author

Note (unrelated to PR): GitHub was acting weird for me, that's why this took so long, and I took this issue. I wanted to complete this. Apologies for the delay, @ValuedMammal. Let me know if there are any changes to make. Everything is working fine now.

@nondiremanuel nondiremanuel added the api A breaking API change label Feb 27, 2024
@nondiremanuel nondiremanuel added this to the 1.0.0-alpha milestone Feb 27, 2024
@notmandatory notmandatory removed this from the 1.0.0-alpha milestone Mar 18, 2024
@ValuedMammal
Copy link
Copy Markdown
Collaborator

Approach NACK

@startup-dreamer
Copy link
Copy Markdown
Author

Approach NACK

Got it also it's been stale for soo long closing this PR.

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

Labels

api A breaking API change module-blockchain

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Merge tx_data_traits.rs into chain_data.rs.

5 participants