search

ThePrevailingOne / pe

0 stars 0 forks source link

Wrong example #11

Open ThePrevailingOne opened 2 years ago

ThePrevailingOne commented 2 years ago

add n/Benedict should be add -a n/Benedict image.png

nus-pe-script commented 2 years ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

While we agree that it is a bug, this is a minor inconvenience because the proper usage of the add command is mentioned further below in the UG, with many examples of proper usage. Moreover, the help window and the error message will allow the user to learn proper usage instead. This was not intended a full example as well (add -a has to take in a phone number / email and will not work even with add -a n/Benedict etc), it is a cosmetic issue instead.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: [replace this with your reason]