AY2122S2-CS2103T-W15-3 / tp

MIT License
0 stars 4 forks source link

[PE-D] Duplicate emails are allowed #138

Closed soc-pe-bot closed 2 years ago

soc-pe-bot commented 2 years ago

Feature tested: add

Steps to reproduce:

add n/222 tg/CS2100 G08 e/e0123456@u.nus.edu

add n/333 tg/CS2100 G08 e/e0123456@u.nus.edu

Issue: Two different students with the same email should not be allowed

Suggestions: Since a duplicate check when two students of the same name already exist, allow the duplicate checker to check for when email already exist.

Screenshot 2022-04-01 at 10.59.52 PM.png


Labels: severity.Medium type.FunctionalityBug original: jinnhl/ped#5

hqhqhq1 commented 2 years ago

address in #127