search

yeozongyao / pe

0 stars 0 forks source link

Unable to input 8 digit phone numbers that have a space in them #3

Open yeozongyao opened 2 months ago

yeozongyao commented 2 months ago

image.png

When trying to add the phone number 9898 3456, which is an 8 digit phone number starting with digit 9, I am faced with the issue that this is a non-number/invalid number/contains blank character. However, it is very common notation worldwide to input phone numbers with a space in them. This will cause occasional hindrance to users who are unaware of this formatting, which is additionally not further stated in the user guide.

Hence, I am assigning this a severity of medium.

nus-pe-bot commented 2 months ago

Team's Response

Severity downgraded as usability of the product is not greatly affected.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Unable to input phone number with spaces

In the UG, it was specified that the user needs to input an 8-digit phone number, however there is no indication that there shouldn't be any space. The app, however, cannot receive any blank spaces.

image.png

[original: nus-cs2113-AY2324S2/pe-interim#1146] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Regardless of the spacing issue, the product is still usable, and numbers are still able to be input into the program. Furthermore the spaces are generally printed to improve readability when showing it to human readers, not when entering data. Thus relabelling as not in scope.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] 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]