search

songivan00 / pe

0 stars 0 forks source link

Adding of multiple same fields take the last field #6

Open songivan00 opened 1 year ago

songivan00 commented 1 year ago

I believe that adding of multiple of the same fields in the add command should not be allowed and an error message should be thrown.

Steps to reproduce: 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/KI n/Song Ivan

Actual output : Shown in the picture below Screenshot 2022-11-11 at 4.52.59 PM.png

Expected output: Error as you cannot have to of the same fields (else should at least state in the UG which field will be taken)

soc-pe-bot commented 1 year ago

Team's Response

It is already stated in our UG that if such parameter is mentioned twice, the last one will be taken.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]