search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Able to edit dentist name to be same as patient name #1782

Open nus-se-script opened 1 year ago

nus-se-script commented 1 year ago

image.png

How to replicate: add-dentist n/John Doe p/98765432 s/Orthodontics y/5 followed by edit-dentist 8 n/Alex Yeohh

Actual: There is both an "Alex Yeohh" in patient list and dentist list, which has the same name but with other fields that are different.

Expected: 1) If you are allowed to have the same person as a patient and a dentist, there should be a warning/confirmation to ask if you wish to enter the same person as a patient and dentist so that all the other fields (phone/address/email etc) are also copied over so they are the same and can clearly reflect the same person 2) If you are not allowed to have the same person as a patient and dentist, then there should be a warning that disallows the user to change a person's name to be both a patient and dentist.

[original: nus-cs2103-AY2324S1/pe-interim#1375] [original labels: severity.Low type.FeatureFlaw]

SynapseProgramming commented 1 year ago

Team's Response

No details provided by team.

Duplicate status (if any):

--