search

CelestineTan03 / pe

0 stars 0 forks source link

Inconsistent error messages #5

Open CelestineTan03 opened 12 months ago

CelestineTan03 commented 12 months ago

image.png

image.png

image.png

steps to reproduce: enter the following commands: add n/ p/999 e/am@gmail/com find n/ delete n/`

for the delete command, the message states that no person found instead of giving the error message that name cannot be empty

nus-se-bot commented 11 months ago

Team's Response

Each function produces a customised error message so the user knows how to change their input such that it is accepted. As such, we feel that this should not be penalised to discredit the extra work put in to craft these error messages. Additionally, for messages like the delete function, we had produced the error message in our UG.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]