Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Student Referenced using Index is not updated after using List #1

Open
Decaxical opened this issue Apr 16, 2022 · 1 comment
Open

Comments

@Decaxical
Copy link
Owner

Decaxical commented Apr 16, 2022

While it has been stated in the User Guide, given the scenario where the TA wishes to enrol a student after using the find command, having to list student again in order to find the index of the student in order to enrol them is too troublesome and diminishes the usefulness of the find command.

@nus-pe-bot
Copy link

nus-pe-bot commented Apr 19, 2022

Team's Response

The intention of the find command is to let the TA easily find a particular student that they may be teaching this semester. As they can be teaching across different modules and classes, this find command will help them easily locate a particular student given the name of the student. Based on our use case stated in our DG, finding students by name would benefit the TA by allowing them to keep in contact with the students.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: Sorry if I caused any misunderstandings, while I understand that the intention of the find command is only for locating students. I believe that it is a FeatureFlaw because the Student Index that all commands reference is the index of the original student index as displayed by list student rather than the currently displayed list of students as shown by commands such as find. This can make it confusing for the users and also makes it more inefficient for the users as they would have to scroll through the list to find the index of the student they wish to run operations on. While the users have other means of identifying the student other than index like student ID. It would be a lot faster and more convenient if they were allowed to use the index of the currently displayed list.

I stated that it "diminishes the usefulness of the find command" because it had the potential to be more useful.


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants