search

ncmathan / pe

0 stars 0 forks source link

Specificity of NRIC error message insufficient #3

Open ncmathan opened 10 months ago

ncmathan commented 10 months ago

The NRIC error message shows the format, but does not specify the number of digits to include between the 2 alphabets.

image.png

nus-se-bot commented 10 months ago

Team's Response

Thanks for pointing this out. The ED doctor receives the IC number to enter from the documentation itself, if the doctor were to make a typo, the error message will appear. However it would prompt the ED doctor to check the IC number again to see if he has inputted correctly. Having an error message that says there should be 7 digits (as mentioned in our UG) will not aid the doctor in any better way since they clearly know it is a typo. This would only be useful if the ED doctor was keying in an IC number from memory. On top of that, we agree typo do happens but this message does not significantly impact their performance.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: " Having an error message that says there should be 7 digits" would definitely help the ED doctor as it specifies exactly what could have caused the typo, instead of the ED doctor figuring it out himself which would take longer and waste their precious time.

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]