-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fee calculation and presentation #621
Conversation
ca4731f
to
b0f2235
Compare
798496c
to
b21d7d0
Compare
b21d7d0
to
b51444c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase or create a new PR on top of main.
b51444c
to
cdba2be
Compare
I was about to test but the fee recommendation from mempool is 7satsvb for all; so I'll wait for at least one variation so I can double check the fee chooser is correct unexpected-funds-all-fees-as-7.mp4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I just ran a test here (video attached) the difference from the fee showed and the fee realized was 2 satoshis: Logs
Videofee.calc.mp4 |
cdba2be
to
8082b2e
Compare
Related breez/breez-sdk-greenlight#294
How to test