Customer phone too restrictive

[nixonwidjaja]

• cannot add international numbers (+1, +62) etc. What if the business's client is a foreigner from the US or Indonesia, perhaps?
• cannot add phone numbers more than 8 digits is too restrictive: phone numbers from the US or Indonesia has more than 8 digits I will classify this bug as feature flaw because this bug makes your app not scalable, i.e., not usable by clients outside Singapore. And in your UG, I see that your target user is home-based business owners. But you don't specify the business's clients, which can be American or Indonesian living in Singapore that hasn't changed their phone numbers.

[nus-pe-script]

Team's Response

No details provided by team.

The 'Original' Bug

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

Phone Numbers shouldn't be restricted to Eight Digits

Phone numbers, as specified in the UG, can only be eight digits. Any attempt to add a phone number not consisting of eight digits, for instance: customer add --name Name --phone 123456789 --email abc@def --address address

...is met with the error message:

Phone numbers should only contain numbers, and it should be 8 digits long.

However, phone numbers may not be eight digits long, especially overseas. Hence, is this overzealous input validation? Perhaps a warning could be displayed instead of blocking the input entirely.

---

_{[original: nus-cs2103-AY2324S1/pe-interim#758] [original labels: severity.Low type.FeatureFlaw]}

Their Response to the 'Original' Bug

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

This bug report would be accepted, however, it would be a documentation bug instead as the UG was not specific enough in stating that the rationale for eight digits is because we only want Singapore phone numbers as we are catering only for Singapore customers with Singapore phone number.

Furthermore, our examples all state Singapore addresses and Singapore phone numbers, thus, we always had it in mind that we the customers are from Singapore but did not state it down explicitly.

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)

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---

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

---