Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

'amount' field doesn't support comma's when buying via moonpay #2400

Closed
Josey87 opened this issue Dec 18, 2019 · 0 comments · Fixed by #2451
Closed

'amount' field doesn't support comma's when buying via moonpay #2400

Josey87 opened this issue Dec 18, 2019 · 0 comments · Fixed by #2451
Assignees
Labels
C - Mobile L - Assigned Lifecycle - Somebody is working on this L - WIP Lifecycle - Work in Progress T - Bug Type - Bug: Something isn't working

Comments

@Josey87
Copy link

Josey87 commented Dec 18, 2019

Bug description

The IOS app I use displays a comma in the on-screen keyboard when entering the amount of IOTA you want to order via Moonpay. In my region it is common to use a comma as decimal separator (netherlands). The mask in the amount field can't cope with this input, since it gives NaN for several fields that use this number as input. See screenshot.

Trinity issue

Bug description

A general description of the bug.

Environment Specs

Apple Iphone 8
OS version 13.2
Trinity version: 1.2.0 (95)

Steps To Reproduce

  1. Open trinity, it opens in the balance screen
  2. tap "buy IOTA"
  3. Confirm the trinity account you want to use
    This opens up the screen where you can input the amount of currency you would like to purchase. My keyboard shows a comma (see screenshot). When entering a normal number, the values are updated in realtime. When entering the comma, the fields go to NaN and $0

Expected behaviour

The software should be capable of processing the given value with the buttons that are supplied on the on-screen keyboard.

  1. Display only a dot or a comma, if this is possible
  2. Convert a comma to a dot in the background
  3. Make the calculation so that it accepts both inputs as correct

Actual behaviour

NaN shows when a comma is entered

Errors

Screenshot provided

@Josey87 Josey87 added the T - Bug Type - Bug: Something isn't working label Dec 18, 2019
@rajivshah3 rajivshah3 self-assigned this Dec 29, 2019
@rajivshah3 rajivshah3 added the L - Assigned Lifecycle - Somebody is working on this label Dec 29, 2019
@rajivshah3 rajivshah3 added C - Mobile L - WIP Lifecycle - Work in Progress labels Jan 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C - Mobile L - Assigned Lifecycle - Somebody is working on this L - WIP Lifecycle - Work in Progress T - Bug Type - Bug: Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants