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

Prefix for change quantity is a/ instead of q/ can confuse user #6

Open
yu-ming-chen opened this issue Nov 13, 2020 · 1 comment
Open

Comments

@yu-ming-chen
Copy link
Owner

I was unsure if I was editing the corret field when excecuting this command.
Screenshot 2020-11-13 at 4.52.11 PM.png

Should have standardise to q/. Marked it as medium as it really added inconvience.

@nus-se-bot
Copy link

nus-se-bot commented Nov 18, 2020

Team's Response

Quantity and amount are different parameters as amount has a sign before the number, while quantity does not. To differentiate the two parameters, we used different prefixes. The difference of the prefixes are also defined in the User Guide as shown below. We downgraded to VeryLow as this is purely cosmetics and does not affect the usage for the user.

image.png

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your explanation]


❓ Issue severity

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

  • I disagree

Reason for disagreement: [replace this with your explanation]


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