ZhiWei1010 / pe

0 stars 0 forks source link

Wrong Error Message for add command #2

Open ZhiWei1010 opened 5 months ago

ZhiWei1010 commented 5 months ago

When no field for nusnet id is given but name field is provided, the error message is 'Names should be in....' and no indication for empty nusnet id is present. This might give confusion to users when they have provided a name but empty nusnet id.

Steps to reproduce: [Assuming running from fresh sample data]

  1. execute addstu n/Alice lu nn/

image.png

nus-pe-bot commented 4 months ago

Team's Response

This error message is correct as the user provided name is not in proper case: (a b is not in proper case but A B is)

Therefore, this issue is rejected

Side note: the steps to reproduce does not match the image.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]