search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Able to add non-standard file types to records #481

Open nus-se-script opened 10 months ago

nus-se-script commented 10 months ago

I was able to add .exe files to a patient's medical records. I believe this could be limited in the future to standard file types, such as .pdf, .docx etc.

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

ryanongwx commented 10 months ago

Team's Response

We did not limit the file type as they are many variations to patient documents. While adding the .exe file may not be typical, the attaching file feature only serves as a form of organising files relevant to patients and what the user decides to attach is up to them. Furthermore, attaching the .exe file does not break the app nor cause any inconvenience to the user and hence we do not feel that it is an issue or a bug.

Duplicate status (if any):

--