Thanks to visit codestin.com
Credit goes to github.com

Skip to content

ADD: Allow custom fees below 1 sat/vByte#7983

Merged
Overtorment merged 3 commits into
BlueWallet:masterfrom
ojokne:allow-custom-fees-below-1-sat-per-vByte
Jul 13, 2025
Merged

ADD: Allow custom fees below 1 sat/vByte#7983
Overtorment merged 3 commits into
BlueWallet:masterfrom
ojokne:allow-custom-fees-below-1-sat-per-vByte

Conversation

@ojokne

@ojokne ojokne commented Jul 1, 2025

Copy link
Copy Markdown
Member

Fixes #7964

This PR removes the restriction that prevented users from entering custom transaction fee rates below 1 sat/vB

Screenshot with custom fee set to 0 sat/vByte

image

Screenshot with custom fee set to 0.5 sat/vByte

image

@Overtorment

Copy link
Copy Markdown
Member

oh ok.

@ncoelho by default such transactions wont broadcast, they will show alert with error that rejectred by network rules or smth. do we need to show more explanation to the user about this? say, show extra warning if user chose feerate between 0.0 and 1.0 ?

@Overtorment Overtorment requested a review from limpbrains July 4, 2025 12:00

@limpbrains limpbrains left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We need a warning and explanation

@Overtorment

Copy link
Copy Markdown
Member

We need a warning and explanation

agree. @ncoelho your input on this?

@ojokne

ojokne commented Jul 4, 2025

Copy link
Copy Markdown
Member Author

We need a warning and explanation

Thank you for your review, I will add the warning and explanation as requested

@Overtorment Overtorment requested a review from ncoelho July 5, 2025 12:25
@ojokne

ojokne commented Jul 6, 2025

Copy link
Copy Markdown
Member Author

We need a warning and explanation

I have added the warning and a simple explanation, Let me know if you would prefer a more specific message or any adjustments.

image

image

@ojokne ojokne requested a review from limpbrains July 6, 2025 18:22
Comment thread screen/send/SendDetails.tsx Outdated
@ojokne ojokne force-pushed the allow-custom-fees-below-1-sat-per-vByte branch from 2762424 to f8ff2b8 Compare July 8, 2025 02:06
@ojokne ojokne requested a review from limpbrains July 8, 2025 02:09

@limpbrains limpbrains left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We should probably adjust policy of BW node to allow such transactions

@Overtorment

Copy link
Copy Markdown
Member

We should probably adjust policy of BW node to allow such transactions

we can accept, but iirc it wont propagate and wont reach miners. such transactions currently can work only when you give it to miner and he mines it

@ncoelho

ncoelho commented Jul 9, 2025

Copy link
Copy Markdown
Member

Looks good to me.

@Overtorment Overtorment self-requested a review July 9, 2025 12:48
@GladosBlueWallet

Copy link
Copy Markdown
Collaborator

Wake the fuck up samurai, we have PRs to merge

image

[all PRs for @ncoelho] https://github.com/BlueWallet/BlueWallet/pulls/review-requested/ncoelho

@Overtorment Overtorment merged commit 13d53eb into BlueWallet:master Jul 13, 2025
11 of 13 checks passed
@ojokne ojokne mentioned this pull request Sep 3, 2025
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.

Allow custom fees below 1 sat/vB

5 participants