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

[BUG] Localize decimal formatting & amount input #119

Closed
ILIYANGERMANOV opened this issue Nov 13, 2021 · 1 comment · Fixed by #131
Closed

[BUG] Localize decimal formatting & amount input #119

ILIYANGERMANOV opened this issue Nov 13, 2021 · 1 comment · Fixed by #131
Assignees
Labels
bug Something isn't working improvement Existing functionality to be improved user request Feature/improvement requested by an user

Comments

@ILIYANGERMANOV
Copy link
Collaborator

Describe the bug
" User feedback:
Hi,
I would like to hide decimal points.
Decimal points don't reflect local number settings
Still see go premium even bought it.
If you need more info please let me know.
"

To Reproduce
Steps to reproduce the behavior:

  1. Set currency to CZK
  2. Input 9000 CZK as the amount
  3. See error
    Error: 9000 CZK should be formatted as "9 000" instead of "9,000"

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
Screenshot_2021-11-13-12-26-56-194_com ivy wallet
Screenshot_2021-11-13-12-27-17-596_com ivy wallet
IMG_20211113_122634

Solution

Use localized grouping separator. For CZK (space) should be used instead of ,.

@ILIYANGERMANOV ILIYANGERMANOV added bug Something isn't working user request Feature/improvement requested by an user urgent labels Nov 13, 2021
@ILIYANGERMANOV ILIYANGERMANOV self-assigned this Nov 13, 2021
@ILIYANGERMANOV
Copy link
Collaborator Author

Format XXX XXX, XX (thousands by space, decimal by comma)

@ILIYANGERMANOV ILIYANGERMANOV added important improvement Existing functionality to be improved labels Nov 13, 2021
@ILIYANGERMANOV ILIYANGERMANOV changed the title [BUG] Amount input grouping separator localization [BUG] Localize decimal formatting & amount input Nov 15, 2021
This was referenced Nov 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working improvement Existing functionality to be improved user request Feature/improvement requested by an user
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant