ivanleekk / pe

0 stars 0 forks source link

TA names may not be fully alphanumeric #10

Open ivanleekk opened 10 months ago

ivanleekk commented 10 months ago

not all names are alphanumeric, some contain / or - or ,. in these cases i cannot enter the full name of the ta into the app. while a minor issue as you can remove those characters while inputting, if you were to have multiple reference lists, some with the ta names containing special characters and others not, it can make sorting through them troublesome

nus-pe-bot 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]

App rejecting valid names with slash

Screenshot 2023-11-17 at 4.38.43 PM.png

Bala s/o Vikram is a valid national registered name, however the app does not allow this TA to be added. While we can work around this issue by perhaps removing the '/', this would not be the best as such names are valid, and the app should be able to accept all valid names without having to go through the trouble of removing certain characters.


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

Their Response to the 'Original' Bug

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

This is the example given in the guideline. However we think the severity should be low.

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]