nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Medical History feature does not allow teachers to track their students' medical history as defined in the scope #4802

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

The introduction in the User Guide states the following:

It also supports keeping track of upcoming meetings and recording the medical histories of students.

However, as the medical command overwrites the medical history of a student if the student already has an existing medical history, there can only ever be the current medical status recorded. Then, the teacher cannot use NewAddressBook to truly track students' medical histories.

image.png (Output shown after inputting medical 1 m/fever, medical 1 m/cold, medical 1 m/flu, showMedical 1)


[original: nus-cs2103-AY2122S1/pe-interim#4764] [original labels: type.FeatureFlaw severity.Medium]

junghon3709 commented 3 years ago

Team's Response

Hi, thank you for your feedback. However, this is the intended behaviour. We said "record" the medical histories, not to keep "track" of them. Moreover, we don't expect many edits to the medical history of the student since the medical history is meant to to store long term major illnesses in the student. It's also explictly mentioned in the UG.

1.png

Duplicate status (if any):

--