nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

(UG) Invalid example in Add command #1809

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Tried adding a patient with the sample example given in UG:

add t/patient n/John Doe o/kidney b/A+ tt/1,2,3,4,5,6 p/98765432 pr/high ic/S9988330G d/S2235456D a/29

However, received the error message as shown below.

Screenshot 2019-11-15 at 4.25.48 PM.png

May want to look into the examples given to ensure it is working


[original: nus-cs2103-AY1920S1/pe-interim#1818]

iskandarzulkarnaien commented 4 years ago

Team's Response

Accepted but downgraded severity to low (as error message states that the NRIC must be 1. valid and 2. have a valid checksum, so there is not TOTAL confusion). Assigned responsibiliy to authors of add command and checksum validation

Duplicate status (if any):

--

iskandarzulkarnaien commented 4 years ago

Team's Response

Root issue is failure to update documentation. Responsibility falls to authors of checksum validation and add command.

Duplicate status (if any):

--