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.

Sign up for GitHub

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

Jump to bottom

Transaction Details dialog should display exact amount of BTC sent. No more. No less. #5305

Closed
wallclockbuilder opened this issue Mar 14, 2021 · 3 comments · Fixed by #5333
Closed

Transaction Details dialog should display exact amount of BTC sent. No more. No less. #5305

wallclockbuilder opened this issue Mar 14, 2021 · 3 comments · Fixed by #5333
Assignees
@wallclockbuilder
Labels
a:bug in:gui

Comments

@wallclockbuilder
Copy link
Contributor

Description

Confirmation for BTC sent displays 0.00BTC sent.
This is because the BTC sent has more than 2 decimal places.

Version

v.1.5.9

Steps to reproduce

Send a Bitcoin transaction with an amount that has more than 2 decimal places e.g. 0.00001234BTC.

Expected behaviour

Confirmation dialog should display exactly what I sent.

Actual behaviour

sending confirmation shows ast 0 00BTC when BTC sent is has more than 2 decimal places It should display full amount

Confirmation dialog displays 0.00BTC Sent.
@BtcContributor
Copy link
Contributor

This happened to me yesterday on Mainnet and I wanted to fix it.

However, I am not able to reproduce it on Regtest v1.5.9, although I am following your steps.

@wallclockbuilder
Copy link
Contributor Author

Please assign to me.

@BtcContributor BtcContributor mentioned this issue Mar 19, 2021
Closed
@ghost
Copy link

ghost commented Mar 19, 2021 •
edited by ghost
Loading

I am testing a fix.
[Update] Fix tested & ready.

@ghost ghost mentioned this issue Mar 19, 2021
Merged
@wallclockbuilder wallclockbuilder mentioned this issue Apr 3, 2021
Closed
@wallclockbuilder wallclockbuilder mentioned this issue Aug 1, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wallclockbuilder wallclockbuilder

Labels
a:bug in:gui
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix race condition in BTC sent confirmation popup
3 participants
@ripcurlx @wallclockbuilder @BtcContributor