Jo3LW / pe

0 stars 0 forks source link

Same person can be added #6

Open Jo3LW opened 1 year ago

Jo3LW commented 1 year ago

Steps:

1) Type add n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 c/3.50/4.00 g/male u/Nanyang Polytechnic gd/05-2024 m/Computer Science ji/173296 jt/Software Engineer Intern t/rejected t/KIV 2) Type add n/John Doe p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 c/3.50/4.00 g/male u/Nanyang Polytechnic gd/05-2024 m/Computer Science ji/17329 jt/Software Engineer Intern t/rejected t/KIV 3) Type view 1 4) Type view 2

Although they have different application ID, all the other fields are the same, even address. Thus most probably they are the same person but the user has inputted the ID wrongly. Perhaps a warning or error can be thrown to inform users that they might have added a duplicate , or typed a field wrongly.

Screenshot 2022-11-11 at 4.42.13 PM.png

Screenshot 2022-11-11 at 4.42.24 PM.png

soc-se-bot commented 1 year ago

Team's Response

It is expected behaviour of the app, where the user can input a completely same person with only the email or Job ID being different. This is to handle the case where applicants are applying to more than 1 roles.\

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]