bryanyee33 / pe

0 stars 0 forks source link

Only 1 phone number is allowed #9

Open bryanyee33 opened 4 months ago

bryanyee33 commented 4 months ago

image.png

Phone numbers only allow 1 phone number to be included, while users might wish to input multiple phone numbers. Eg. Secondary phone number if first one fails, phone numbers of next of kin, etc.

Such details should be allowed to be included instead of overzealous input validations.

nus-pe-bot commented 4 months ago

Team's Response

We can consider it in future iterations.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: This is the exact same scenario as Example 1 given for overzealous input validation with phone numbers.

image.png

Moreover, the devs placed this issue under response.NotInScope, with the reasoning of "We can consider it in future iterations.", even though there is no mention of fixing this issue in the UG or DG in the future. Also, there was no explanation on whether rectifying this is less important. Hence, their reasoning is invalid and insufficient, and there is no justification provided for putting it under response.NotInScope.

image.png


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** There is no explanation provided for why this was downgraded. Moreover, `severity.Medium` is reasonable, as this app is meant to "facilitate seamless management of academic contacts and data" for academic instructors, making it likely that they would need multiple phone numbers such as next-of-kin in case of emergencies, etc.