Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ensure we dont create BTC transaction outputs below the bitcoin dust limit #4340

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

martin-chainflip
Copy link
Contributor

@martin-chainflip martin-chainflip commented Dec 12, 2023

Pull Request

actually closes PRO-855 😁

Checklist

Please conduct a thorough self-review before opening the PR.

  • I am confident that the code works.
  • I have updated documentation where appropriate.

Summary

Please include a succinct description of the purpose and content of the PR. What problem does it solve, and how? Link issues, discussions, other PRs, and anything else that will help the reviewer.

Copy link

codecov bot commented Dec 12, 2023

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (e8fd491) 71% compared to head (d2f7a6d) 71%.

Files Patch % Lines
state-chain/chains/src/btc/api.rs 0% 13 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##            main   #4340    +/-   ##
======================================
- Coverage     71%     71%    -0%     
======================================
  Files        390     390            
  Lines      66407   66260   -147     
  Branches   66407   66260   -147     
======================================
- Hits       47451   47232   -219     
- Misses     16580   16652    +72     
  Partials    2376    2376            

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@martin-chainflip martin-chainflip merged commit 869e848 into main Dec 12, 2023
41 checks passed
@martin-chainflip martin-chainflip deleted the fix/btc_utxo_dust_limit_checks branch December 12, 2023 17:13
syan095 added a commit that referenced this pull request Dec 18, 2023
…-timeout

* origin/main:
  feat: end to end network upgrade github action (#4274)
  feat: spec_version of PR is greater than spec version of current release (#4355)
  feat: enforce version is greater than release version on PRs to main (#4351)
  fix: btc witnesser test failing sometimes (#4353)
  fix: connections can become stale when reconnecting (#4310)
  chore: add `chainflip-rpc-node` systemd file 🚀 (#4352)
  feat: API Bins check SC compatibility (#4342)
  chore: update runtime spec checks ⛓️ (#4349)
  feat: Add version cmd to all bins (#4343)
  fix: changelog check 🤫 (#4348)
  chore: update docker tags 🐳 (#4347)
  chore: add runtime version check 👀 (#4344)
  feat: shave fees on ingress (#4335)
  pick/persa fixes (#4329)
  feat: track btc fees on success (#4334)
  ensure we dont create BTC transaction outputs below the bitcoin dust limit (#4340)
  fix: sweeping before withdrawal (#4337)
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.

None yet

3 participants