Phone Number field restriction

[wang-h-z]

Description

The p/ prefix must only be exactly 8 digits long, begin with 6, 8, or 9, and contain only numeric characters without spaces. Numbers that only exist in Singapore's context. Attempting to add a character such as '+' for area code is also not allowed. This causes some hinderance the users since a valid Singaporean number is +65 97562222 is a valid number and numbers from international users will not be able to entered into the application.

---

Steps to Reproduce

1. [Step 1: Attempt to add a client with a phone number greater than 8 digits in the p/ prefix such as p/1234567890]

Additional Information

• Environment: macOS

• Screenshots: Attach relevant screenshots or videos here.

• Logs: Include any relevant log output here, if applicable.

  INFO: Invalid command: Invalid command format! Phone numbers must be exactly 8 digits long, begin with 6, 8, or 9, and contain only numeric characters without spaces. add: Adds a client to the address book. Parameters: n/ p/ e/ a/

  j/ i/ [t/ ]... [r/ ]...[s/ ]... Example: 'add n/ John Doe p/ 98765432 e/ johnd@example.com a/ 311, Clementi Ave 2, #02-25 j/ doctor i/ 300 t/ GOLD r/ He is very smart s/ NON_URGENT'

Suggested Improvement

Perhaps allow for less restrictive parameters for the phone number to accomodate for international clients.

[soc-pe-bot]

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Thank you for your bug report. After reviewing your submission, we have decided to reject the report. As outlined in our User Guide under Section 5.4.1, "Flag Arguments, our application only accepts valid Singaporean personal phone numbers.

Since all supported phone numbers are Singaporean, there is no need to include the '+' prefix. Additionally, our application is designed exclusively for credit card sales to local users, and we do not support international phone numbers or users at this time.

We appreciate your feedback and encourage you to reach out if you have any further concerns or suggestions.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

• [ ] I disagree

Reason for disagreement: [replace this with your reason]

---

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

---

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

---