yespiggy / pe

0 stars 0 forks source link

Incomplete Error Message for Add Command #9

Open yespiggy opened 2 months ago

yespiggy commented 2 months ago

image.png

The error message in the app for add command does not include s/ and l/ as stated in the UG. Users who did not refer to the UG might never realize these two fields also exists.

Also the error message is missing spaces here: nt/NOTE[ap/APPOINTMENT][t/TAG]

nus-se-script commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Inconsistent format in TutorRec and in UG

Upon executing the add command, the error message shows a format requires all fields p/PHONE, e/EMAIL, etc as compulsory, and only ap/ and t/ are optional. This is in contrast to the format put in the UG, which states that only name is compulsory and the rest are optional.

Application: image.png

UG: image.png

Steps to replicate:

  1. Try keying in the command: "add"

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

Their Response to the 'Original' Bug

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

Agreed

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`] - [x] I disagree **Reason for disagreement:** ![image.png](https://raw.githubusercontent.com/yespiggy/pe/main/files/fa2415d8-66f5-4eb1-aa71-b81aabe66760.png) I think the issue severity should be medium, as the problem is similar to this one in the canvas quiz. For users who do not refer back to the UG, they might never realize that they can add `s/` subject and `l/` levels as fields too.