tayziyi / pe

0 stars 0 forks source link

Lack of limit for phone number #8

Open tayziyi opened 2 months ago

tayziyi commented 2 months ago

image.png

To replicate: add n/Alex Yeohh p/874388079999999999999999999999999 e/alexyeoh@example.com a/Blk 30 Geylang Street 29, #01-40 t/supplier t/durian Based on research, app should be able to support it. (3 - 15 digits) Indonesia currently has the longest phone number, 12 digit (but allowed 15 including country code). Shortest is 3 digit subscriber numbers As such, there should definitely exist a check for maximum number of digits of a phone number.

nus-pe-script commented 2 months ago

Team's Response

image.png

According to the textbook, no harm is brought to our app or data when a long phone number is entered. Therefore, this is not a bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


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