nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Finance Tracker: edit-install doesn't work with large numbers #1579

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Expected:

Amount to be changed to $10000000000000000000000000000000000000000000000000000000001

Actual:

Amount stays the same at $100000000000000000000000000000000000000000000000000000000000

Steps to reproduce

  1. edit-install 1 a/100000000000000000000000000000000000000000000000000000000000

Screenshot 2019-11-15 at 4.50.15 PM.png

  1. edit-install 1 a/10000000000000000000000000000000000000000000000000000000001

Screenshot 2019-11-15 at 4.50.04 PM.png


[original: nus-cs2103-AY1920S1/pe-interim#1588]

gohsnn commented 4 years ago

Team's Response

Noted. Similarly, the issue would unlikely to affect normal operations of the product as Jarvis is meant to track student expenses and it is unlikely a student would spend $10000000000000000000000000000000000000000000000000000000001 for a monthly installment.

Duplicate status (if any):

Duplicate of #2148