nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Unable to add more than 8 medical histories #3459

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Expected:

• Expected Behaviour: I feel like I should be able to add more than 8 records, especially since the UI can expand. No justification was provided in the UG as well.

Actual

• Actual Behaviour: Maximum 8 records are allowed. • Why that severity? It causes a minor inconvenience and is rare that patients will have so many disorders.

Steps to reproduce:

  1. Add 8 medical histories.
  2. Add a 9th.

Screenshots:

Screenshot 2021-11-12 at 4.17.25 PM.png


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

didymental commented 3 years ago

Team's Response

Thank you for the feedback.

Our team had an intense debate over this as well. We were balancing out how the UI would look for a patient with too many medical entries and how many medical entries are actually sufficient.

Ultimately, we came to a conclusion that 8 is sufficient and it would balance out how the UI would look for a patient. Our decision was not based on intuition:

  1. Our target users are small family clinics -- patients visiting these clinics do not have that many or serious conditions that need to be taken note of. If they do, they should go to the hospital instead, which is not an intended target user for us.
  2. Only a third of the world's population has more than 5 ailments. This means on average, we can expect that an average patient visiting a small family clinic to have less than 5 ailments to record. We provided up to 8 as sufficient buffer for this.

Screenshot 2021-11-13 at 1.40.08 PM.png

Hence, we reject this issue.

Duplicate status (if any):

--