search

AY2324S1-CS2103T-T08-1 / tp

MIT License
0 stars 5 forks source link

patient bugs #172

Closed derekjxtan closed 10 months ago

derekjxtan commented 10 months ago

Tested up to edit_p based on user guide.

  1. add_p error message missing newline before parameters and example.

    image

  2. add_p invalid phone numbers are accepted. (problem with phone validation, will move out of here)

    image

  3. edit_p invalid command error message parameters. What does ... mean? Nric given in example also not valid.

    image

  4. edit_p success message fields order differs from user guide. User guide shows in this order: name, nric, phone

    image
yanghengtang commented 10 months ago

Update:

Pointer 1 resolved in #179

yanghengtang commented 10 months ago

point 3 still not resolved

yanghengtang commented 10 months ago

all issues resolved