nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Unable to edit student ID #5186

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

The "edit student" command of the app does not provide a functionality to edit student ID. Thus if the user made a typo in student ID and later discovered it, the user must delete the student entirely, and add a new student with the correct ID. This forces the user to re-type all student information including name, email and tele handle, even if they are all correct in the original student.

Suggestion: I understand that ID is used to identify a student, but the problem is this implementation does not allow the editing of ID itself. Perhaps you can identify students by their index number listed in the mod. In this way, identification is not dependent on any editable data.

Edit student.png "edit student" command does not provide functionality to edit student ID, according to UG


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

tanjoewel commented 2 years ago

Team's Response

Student ID is being used as an identifier hence it is crucial in keeping the identity of the student separate. Student ID is meant to be uneditable.

Duplicate status (if any):

--