search

Goh-Li-Ting / pe

0 stars 0 forks source link

Unclear priority for find command #8

Open Goh-Li-Ting opened 10 months ago

Goh-Li-Ting commented 10 months ago

Found by NRIC when name and nric are the same. While it is highly unlikely for NRIC and the NAME to be the same, the find function for nric and name should be separated in case they are the same.

Before find: image.png

After find S8811111Z: image.png

Found by gender when name and gender are the same.

Before find: image.png

After find F: image.png

nus-se-bot commented 10 months ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Add patient with name M

Screenshot 2023-11-17 at 5.09.23 PM.png

How will find M work in this case?

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

Their Response to the 'Original' Bug

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

Please refer to the following sections in the UG:

Screenshot 2023-11-19 at 10.44.21 AM.png

Screenshot 2023-11-19 at 10.44.35 AM.png

Screenshot 2023-11-19 at 10.46.42 AM.png

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.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]