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

ui: (wallet, transaction history) formatting issue for Amount #2994

Open
norwnd opened this issue Sep 28, 2024 · 3 comments · May be fixed by #2998
Open

ui: (wallet, transaction history) formatting issue for Amount #2994

norwnd opened this issue Sep 28, 2024 · 3 comments · May be fixed by #2998

Comments

@norwnd
Copy link
Contributor

norwnd commented Sep 28, 2024 •
edited
Loading

Running 1.0.0, there is a formatting issue under Polygon Transaction History section of Wallet tab, namely Amount is showing 0.001 of what it should be (CurrentShownAmount=CorrectAmount*0.001),

same issue with incorrect amount is seen if I double-click into particular transaction record to see more details.

I should note, this is what I observe for USDT/USDC (which have 6 decimal points I believe), not sure whether other currencies are affected.

No screenshots for this one for privacy reasons.
@dev-warrior777
Copy link
Contributor

6 decimal points for USDT confirmed .. it is configurable with most tokens using 18 I believe

@norwnd
Copy link
Contributor Author

norwnd commented Sep 29, 2024

Additional data point: seems send/receive wallet transactions with DCR and MATIC show the correct amount.

@martonp
Copy link
Contributor

martonp commented Sep 29, 2024

Fixing.

@martonp martonp linked a pull request Sep 29, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

app: Fix amounts displayed for token transactions in tx history martonp/dcrdex
3 participants
@martonp @norwnd @dev-warrior777