seantaysl / pe

0 stars 0 forks source link

Undo command does not undo list command. #4

Open seantaysl opened 3 years ago

seantaysl commented 3 years ago

As can be seen below, the undo command does not undo list command.

First, I did find n/alex to list out only alex in the patients window. Note that Alex currently has no remark as I removed it prior to this. No details provided. Screenshot 2020-11-13 at 4.38.04 PM.png

Next, I did list to list out all patients. Screenshot 2020-11-13 at 4.38.32 PM.png

Finally, I did undo to undo the list but the command remark to edit remark was undone instead. This can be seen from Alex having remark "friendly guy" again. Screenshot 2020-11-13 at 4.39.01 PM.png

nus-pe-bot commented 3 years ago

Team's Response

It is mentioned in the UG that undo command only reverses an undoable command (list not included)

Items for the Tester to Verify

: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.High]

[ ] I disagree

Reason for disagreement: [replace this with your reason]