Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to use budget command for values above 10000000 #5

Open
LiuChao93 opened this issue Apr 17, 2020 · 1 comment
Open

Unable to use budget command for values above 10000000 #5

LiuChao93 opened this issue Apr 17, 2020 · 1 comment

Comments

@LiuChao93
Copy link
Owner

When using values higher than 10000000 for budgetting command, e.g. budget b/10000000 rc/, the application does not throw any errors and does not do anything.

Screen Shot 2020-04-17 at 2.38.33 PM.png

@nus-pe-bot
Copy link

nus-pe-bot commented Apr 21, 2020

Team's Response

We purposely made it such that users cannot input numbers more than 1000000000 for budget or transaction value. We forgot to include an error message along with that error. We will fix it in the next release.

Items for the Tester to Verify

❓ Issue severity

Team chose [severity.VeryLow]
Originally [severity.High]

  • I disagree

Reason for disagreement: There is no mention in the UG or DG that the value shoud not be more than 1000000000. Hence, it is presumed that the value could be more than 1000000000. If the value could be more than 1000000000 and the application does not allow it, it is an error that could affect all of the users.

If the value could not be more than 1000000000, assuming it is mentioned in the UG or DG, then without throwing any error messages to the users (in fact the application does not do anything, it does not throw any error), the users do not know it is an error. This affects all of the users.

Hence, the severity should be high.


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants