nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Unclear error message given multiple error input fields #2508

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Steps to replicate:

  1. Launch the app
  2. Edit the 10th person with new birthday "(ab)edit 10 b/06-32"

image.png

Although both of index and date is wrong, the error message only indicates wrong date format. The users may not know the index is also not accepted.


[original: nus-cs2103-AY1920S2/pe-interim#2495]

ElysiaTanZY commented 4 years ago

Team's Response

This is because the birthday field is being parsed first before the app tries to edit the stated person. If the birthday field is already wrong, there is no point checking and trying to edit the stated person.

Duplicate status (if any):

--