Joseph31416 / pe

0 stars 0 forks source link

Phone number does not allow for spaces #3

Open Joseph31416 opened 7 months ago

Joseph31416 commented 7 months ago

Command: addmem n/John Doe hp/9876 5432 e/johnd@example.com a/John street, block 123

Actual: The following error message was given: Phone numbers should only contain numbers, and it should be at least 3 digits

Expexted: This entry should be added in as some people might prefer to have spaces between every 4 digits for readability.

bug 2.png

soc-pe-bot commented 6 months ago

Team's Response

Thanks for the suggestion! The User guide as you quoted says that it only accepts numbers, and we feel that even without the ability to add spaces, it is still as usable of a product.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** Thanks for responding. However, according to the course website, a severity low bug is one "that is unlikely to affect normal operations of the product [and] appears only in very rare situations and causes a minor inconvenience only." (screenshot 1). I would say that this issue does not just occur in very rare situations as there could be a variety of reason for why the user might want to have spaces between phone number, such as for having spaces for every 4 digits like "9999 9999" or having spaces between multiple phone numbers like "12345678 09876543". Furthermore, this causes more than minor inconvenience as there would be no alternative to accomplishing the task of storing multiple phone numbers. As such, I would classify this as a flaw that causes occasional inconvenience to some users, but they can continue to use the product, hence a severity of medium. Screenshot 1 ![Screenshot 2024-04-23 at 12.33.09 PM.png](https://raw.githubusercontent.com/Joseph31416/pe/main/files/9474df52-db8d-4265-8d69-3c90d5100e0d.png)