search

BitCannaGlobal / WeedMobile

5 stars 1 forks source link

"Amount" field when creating a QR needs to only allow numbers #2

Closed TJ-BitCanna closed 9 months ago

TJ-BitCanna commented 11 months ago

Right now it is a "free-for-all" field where you can fill all possible texts/values/characters/....

For being user-friendly it is desired to have this field only support numbers and up to 6 decimals maximum to comply with the BCNA decimals.

atmoner commented 11 months ago

I have added type="number" on amount field Fixed here: https://github.com/BitCannaGlobal/WeedMobile/commit/264920a58889423f69f9b1f1adc1f6661e6d6b06

You can try on next app update

TJ-BitCanna commented 11 months ago

Works on Android! 2023-09-21 12 59 30

bcnajohan commented 11 months ago

The field of the amount should be limited to only 6 numbers after the decimal for both iOS and Android.

atmoner commented 11 months ago

And finaly work on IOS!!

image

Can be tested =)

TJ-BitCanna commented 11 months ago

Works on Android for a while :) But it needs a limitation to 6 decimals on the "Amount" field still (and the same applies for the "Send token" screen as well)

atmoner commented 9 months ago

Normally, support for decimal is already done! Maybe I forgot to update the post Can be tested!

image

TJ-BitCanna commented 9 months ago

Android approved.

TJ-BitCanna commented 9 months ago

Limitation to 6 decimals works on iOS as well.

The fields are not limited to numbers only, but will keep that on #37