lwj1711 / pe

0 stars 0 forks source link

Names with s/o or d/o not allowed #10

Open lwj1711 opened 3 years ago

lwj1711 commented 3 years ago

Since names might contain characters like " ' " and "/", the app should allow for them to be included. In the case of "d/o" :

image.png

we can see that this is not allowed

Problem is magnified with the use of "s/o", since users might forget that it is a prefix and input it, resulting in larger problems.

For example, adding under name "xxx s/o yyy" results in the following:

image.png

nus-pe-bot commented 3 years ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Applicant names doesn't allow '/'

Names of Indians often contain "s/o", which is not allowed in this app.


[original: nus-cs2103-AY2122S1/pe-interim#1439] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

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

Accepted. We did not conduct sufficient research on our target user.

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: Names with "/" is not the only problem. Issue with " ' " is also pointed out.

Example: Sinéad O'Connor


:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: image.png

I believe the severity should at least be Medium, given how many Indians have "/" in their names. It will be very difficult for the users to enter their full names in such cases.

Given how Singapore (their target audience in UG) is multi-rational, most employers will have Indian interviewees, and this will pose a problem for most users. As such, problem might even be a High severity since it affects most users and causes major problems through not being able to type a group of interviewees' full names.