yytan25 / pe

0 stars 0 forks source link

Find by age doesn't accept a range of ages #7

Open yytan25 opened 1 year ago

yytan25 commented 1 year ago

Currently, the find command using the optional parameter a/ can only find a patient with an exact age.

image.png

However, as a user of the app, they may want to find patients of a range of ages instead of just an exact age. This will cause some inconvenience, as usually the exact age is insufficient in meeting a user's needs.

nus-se-script commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Find by age does not allow multiple ages or age ranges

The find feature by age will normally be used to send warnings (about viruses) for babies or adults, for example.

These will require manual searching for each age.

Moreover, patients might not remember their exact age but rather their date of birth. This means a small trial and error process to find by their age needs to be done.

Screenshot 2023-11-17 at 5.01.30 PM.png


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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

The scenario given is very unlikely, but the extension of finding a patient record by an age range is a good suggestion. The implementation of this feature is not in scope of latest release as it is low priority.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]