Skip to content

Conversation

@ddustin
Copy link

@ddustin ddustin commented Sep 23, 2024

Added an explanation for aborting the splice if the feerate is too low

Copy link
Member

@t-bast t-bast left a comment

Choose a reason for hiding this comment

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

I'm not sure this is where the error you're seeing is created, this is only for the case where you try to use a feerate that is below our mempool minimum feerate. I think you're getting an error inside the InteractiveTxBuilder, I'll look into improving the error message.

Added an explanation for aborting the splice if the feerate is too low
@ddustin ddustin force-pushed the ddustin/splicing_feerate branch from 3652211 to 1b28236 Compare September 24, 2024 04:51
t-bast added a commit that referenced this pull request Oct 8, 2024
This can be helpful when troubleshooting cross-compatibility issues.

Replaces #2911.
@t-bast
Copy link
Member

t-bast commented Oct 8, 2024

I think you're getting an error inside the InteractiveTxBuilder, I'll look into improving the error message.

I've done this in #2920, which can replace this PR. I believe the InteractiveTxBuilder case is what you're running into, not the mempool min relay feerate.

@t-bast t-bast closed this Oct 8, 2024
t-bast added a commit that referenced this pull request Oct 18, 2024
This can be helpful when troubleshooting cross-compatibility issues.

Replaces #2911.
t-bast added a commit that referenced this pull request Oct 18, 2024
This can be helpful when troubleshooting cross-compatibility issues.

Replaces #2911.
sstone pushed a commit that referenced this pull request Oct 21, 2024
This can be helpful when troubleshooting cross-compatibility issues.

Replaces #2911.
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.

2 participants