nus-pe-bot
commented
2 years ago Team's Response
We have already mentioned this in the UG.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
Delete command doesn't delete patient(s) from underlying list and only what's displayed
Reproduce
findElderly wincqoievnqeirnoqirgnirng deleteElderly 3This gives me an empty list because the search itself is gibberish.
Expected
Supposed to delete the 3rd patient from the raw underlying patient list.
OR the UG must say that they only delete elderly information displayed on the app's UI list.
Actual
Says index is invalid.
[original: nus-cs2103-AY2122S1/pe-interim#930] [original labels: severity.Low type.FunctionalityBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
We have already stated this in the UG.
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)
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
:question: Issue response
Team chose [response.Rejected]
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
:question: Issue severity
Team chose [severity.VeryLow]
Originally [severity.Low]
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
Reproduce
The first command gives an empty list. The second command does not give me the details of patient 2 even though they exist.
Expected
App must show details of elderly patient 2.
OR UG must state that the
viewDetailscommand works only for patients displayed on the list.Actual
Tells me patient 2 doesn't exist even though they do in the underlying data.