mgiang2015 / pe

0 stars 0 forks source link

Undo command does not work after exiting application #4

Open mgiang2015 opened 3 years ago

mgiang2015 commented 3 years ago

After exiting application, the last command executed cannot be undone when application is started again. This is not mentioned in Documentation.

Steps to reproduce:

  1. Execute p-add n/John Doe123 g/MALE bd/2018-12-27 bt/A+ p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 t/friends t/owesMoney
  2. exit
  3. Open application again and run undo.

Expected:

  1. JohnDoe123 patient deleted.

Result:

  1. Nothing to undo
nus-pe-bot commented 3 years ago

Team's Response

Accepted.

Thank you for your feedback. However, we believe that it is a common practice for many applications to get rid of any user’s history once they exit the application or else your computer may require a ton more memory space to store “junk” history after exiting any application. Therefore, undo is a command that is resetted after every exit of the application. This could be clarified more in the UG, we apologise for the gap in information.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Team response said that this bug is accepted.


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: I personally think severity should be Low because this flaw does affect usage in rare occasions, for example, after my patient cancelled his/her appointment, I deleted the appointment and exited the app. However 5 minutes later, my patient managed to reschedule his/her other plans and wanted to keep the appointment. In this case, if I knew that I could not undo, I would ask for all the information related to the appointment again. However, if I assumed I could undo, I would hang up the phone and try to undo my previous changes without asking for details, which will cause inconvenience for me.