peiran18 / pe

0 stars 0 forks source link

Name can only contain Alphanumeric characters #1

Open peiran18 opened 10 months ago

peiran18 commented 10 months ago

image.png

Valid names such as Abdul s/o Muhammad and Mary-Jane cannot be input due to the constraints. This is a slight inconvenience that users may face when inputting contacts with such names.

nus-pe-script commented 10 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Cannot add people with names containing S/O

S/O is a common addition to certain names, but app throws an error when trying to add people with s/o in their names. As a significant number of people have this in their name, and the app is designed for professional use and therefore is likely to require saving full names, it is a problem when trying to save full names.

image.png


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

Their Response to the 'Original' Bug

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

Thank you for bringing this to our attention!

In the UG, we wrote this:

image.png

Hence the feature works as intended. But we agree that it is not that well designed and hence it is a feature flaw.

However, we believe that the severity is low. In the context of SOCareers, it is not too important to save the contact name exactly like how it's spelled (e.g. compared to a hospital records management system, for example). So the user can do a workaround of "Ramesh so Raj".

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 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]