search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Unable to add student names with special character '/' #3229

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

To reproduce: add n/Raja S/O Bala p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 t/friends t/owesMoney i/A0123456G h/@good_student c/CS2030S ta/CS1101S cg/CS2030S:Tut07

Expected: Student is added

Actual: Names should only contain alphanumeric characters error shown

This can be quite a common name for students and not allowing such input can slightly inconvenience the professor. Screenshot (50).png

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

leongdl135 commented 2 years ago

Team's Response

This is a tricky issue since our application only supports addition for alphanumeric characters in the name.

image.png

There are many unique names in the world for example, Elon Musk's sons name is X Æ A-12. By trying to accommodate all sorts of characters, it would be hard as certain computers and operating systems cannot display certain characters.

Most importantly, in ProfNUS, we are identifying students based on their unique Student ID. As such, the name does not necessarily have to be the full name of the student and is not expected to match the legal name of the person, but rather just something convenient for the user to remember. As such, we believe that this is not a feature flaw, but rather an intended outcome for our application.

Duplicate status (if any):

--