parasytezz / pe

0 stars 0 forks source link

Wrong error message for add-expense #3

Open parasytezz opened 4 days ago

parasytezz commented 4 days ago

The amount that was specified in the command is correct according to the User Docs, it is the date command that is wrong as it does not have a space after d/. However the error message specifies that it is an error with the amount Screenshot 2024-11-15 163925.png

nus-pe-script commented 22 hours ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

wrong error message for add-income

The amount that was specified in the command is correct according to the User Docs, it is the date command that is wrong as it does not have a space after d/. However the error message specifies that it is an error with the amount image.png


[original: nus-cs2113-AY2425S1/pe-interim#204] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for the report, we have specified in the Document that the keyword must be followed by a space as it is expected to read all the string after a keyword until the end of the input string/ next keyword found. We've also printed out the whole value read so that the user can notice that the command went wrong but yes, the message may sound confusing! However, we have lowered the severity to Low as we do not believe this bug to be very misleading to the user given that we have specified in the user guide that a space must follow all keywords, and should only cause a minor inconvenience to users who may have wrongly input the command.

Thank you!

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]