search

lyeyixian / pe

0 stars 0 forks source link

Unclear error messages #5

Open lyeyixian opened 3 years ago

lyeyixian commented 3 years ago

editPerson with pid value "-1" or "0" shows error messages "No existing personId can be found." which is unclear. Seems like -1 and 0 is a valid value for pid.

image.png

image.png

editPerson with large pid value shows

image.png

but the value is a positive integer.

nus-pe-bot commented 3 years ago

Team's Response

This error message still does not affect the usage of the user. This is still a valid error message (i.e it is true that no existing personId has the id of -1).

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]