search

Quasant / pe

0 stars 0 forks source link

Incorrect error message, sn not indicated as optional #8

Open Quasant opened 1 week ago

Quasant commented 1 week ago

Steps to reproduce: Simply type adda in command box.

image.png

image.png

As in the title, error message does not reflect the actual optionality of parameters. This was the most prominent one across the commands.

nus-pe-bot commented 1 week ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Unstandardized parameter choice between User Guide and the App

Screenshot 2024-11-15 at 4.54.10 PM.png

Screenshot 2024-11-15 at 4.54.20 PM.png

User Guide shows that student number is optional but the app did not mention it.

[original: nus-cs2103-AY2425S1/pe-interim#1026] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

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

The command works as specified in the user guide. However, the error message was ambiguous which is a feature flaw. Thanks for the spot!

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 type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** The error message is not ambiguous, it is simply incorrect. Probably an oversight since there are other commands whose error messages did indicate optional parameters with `(optional)` as well.