ChrisHo1341 / pe

0 stars 0 forks source link

Most commands require the user to specify the student using their student ID #7

Open ChrisHo1341 opened 2 months ago

ChrisHo1341 commented 2 months ago

image.png

Most command such as in the above example "tag" require the user to the specify the student via their student ID. The student ID consists of 9 characters following a specific format. This requirement is quite a hassle for the user and may slow down their efficiency when using EduLink.

The page already has numbering for each student, perhaps could allow the user to specify by the numbering for each student as well such as "1" for Alex Yeoh as shown in the screenshot

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

Required to use student id for some commands

For several of the commands such as tag and grade, it is required for the user to type the student id to modify the contact. This is time-consuming and is prone to errors. Eg. Typing A0269501E instead of A0265901E, thus incorrectly modifying the wrong student, possibly without the user realising. This could have been prevented by allowing users to edit via the index such as the delete command.


[original: nus-cs2103-AY2324S2/pe-interim#2126] [original labels: type.FeatureFlaw severity.High]

Their Response to the 'Original' Bug

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

  • Invalid severity provided.
  • In future iterations, we will allow the commands to modify data using indexes.

    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 response Team chose [`response.NotInScope`] - [x] I disagree **Reason for disagreement:** ![image.png](https://raw.githubusercontent.com/ChrisHo1341/pe/main/files/a1d73438-2fb3-4a3e-9e64-88f2c47df3d0.png) I believe this case falls under the second point as this change will improve the end-user's experience without much additional effort as there are other commands that already use the index