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

TAs can only add existing contacts as students #3558

Open
nus-pe-bot opened this issue Apr 16, 2022 · 1 comment
Open

TAs can only add existing contacts as students #3558

nus-pe-bot opened this issue Apr 16, 2022 · 1 comment

Comments

@nus-pe-bot
Copy link

nus-pe-bot commented Apr 16, 2022

Screenshot 2022-04-16 at 3.06.49 PM.png

This seems like unnecessary complexity since the students in a TAs class need not necessarily be a person they already know and is in their contact list. However, in order to add a student TAs must also add them as contacts first.

To avoid this the person list could be removed or kept separate from the student list.


[original: nus-cs2103-AY2122S2/pe-interim#3541] [original labels: severity.Low type.FeatureFlaw]
@honganhcs
Copy link
Contributor

Team's Response

Thank you for your suggestion! However, the product is designed in this way so that TAs can keep track of all their contacts as well manage their classes, students and assessments. As students have the same properties as other contacts, it is reasonable to add students to the app as contacts first, before assigning them to classes.

Duplicate status (if any):

--

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