search

dom-buri / pe

0 stars 0 forks source link

Nit Picking: Phone number does not allow more than 8 numbers #4

Open dom-buri opened 9 months ago

dom-buri commented 9 months ago

Some dentist who uses an international number might have more than 8 numbers in their phone number. Hence, perhaps having more flexibility on the phone number will be good.

Screenshot 2023-11-17 at 4.20.13 PM.png

soc-se-bot commented 9 months ago

Team's Response

Refer to #2983

The 'Original' Bug

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

Phone number restricted to 8 digits

Phone number is restricted to 8 digits. However, I dont think the UG mentioned that this app is catered to specifically singapore users. Hence, users for eg having thai telephone number may not be able to use this application. Especially when foreigners may need a quick check for their teeth in emergencies.

image.png

[original: nus-cs2103-AY2324S1/pe-interim#4293] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

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

Thank you for the report. Our group designed the app to specifically target dental clinic administration assistants in the Singapore context, hence, we decided to go with strictly 8 digits for Phone numbers. We do admit that we could have made this clearer in the User Guide and that the implementation could have been better by allowing more flexible input.

We categorised this as a FeatureFlaw rather than FunctionalityBug as the functionality (phone number input) is working as currently designed (accepting 8 digits), but the design itself does not accommodate international use cases (longer inputs or space in inputs), which is why it is a flaw in the feature rather than the functionality.

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 severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]