Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

Error in cost display #109

Open
kvas001 opened this issue Apr 13, 2023 · 3 comments
Open

Error in cost display #109

kvas001 opened this issue Apr 13, 2023 · 3 comments
Assignees
Labels
bug Something isn't working product/console sev2

Comments

@kvas001
Copy link

kvas001 commented Apr 13, 2023

photo_2023-04-14_04-14-24

2 days = 5 AKT
1 month = 75 AKT
15.95 - dollars, not AKT

====

When we fix this, let's also change the way we display values to be:
"[Dollar Value] | [AKT Value]"
e.g $15.95 | 75 AKT

@zJuuu
Copy link
Contributor

zJuuu commented Apr 13, 2023

Thanks for reporting! Will try to submit a PR to fix this asap

@anilmurty
Copy link
Contributor

anilmurty commented Apr 13, 2023

hey @zJuuu - thanks for taking this on, @jtary and I just talked about this earlier today. In addition to fixing the bug where the code takes the AKT value and assumes it is dollars (and then converts that to AKT before displaying both) it will be great if we also changed the way we display all 3 pairs of values in this screenshot to be "[Dollar Value] | [AKT Value]" to be less ambiguous than "[Dollar Value] / [AKT Value]" (which gets confusing with things like "balance" where it looks like we're talking about "this much of that used" rather than "this much in $ or this much in AKT"). Hope that makes sense :) (FYI @aktdenis as well)

@anilmurty
Copy link
Contributor

Updated description/ requirement as well

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working product/console sev2
Projects
None yet
Development

No branches or pull requests

3 participants