search

RadieonAjax / pe

0 stars 0 forks source link

FeatureFlaw: Undo command unintuitive #3

Open RadieonAjax opened 2 weeks ago

RadieonAjax commented 2 weeks ago

Rationale: I understand that currently, help and closeat is not undo-able, but the current implementation is rather unintuitive in the sense that it requires me to execute undo more than 1 time to undo my previous command. The program should be able to check that the user enters commands that cannot be undone, and don't save them in the stack or a variable which contains the last command. It's a great feature, but could be a little better!

Step to reproduce:

  1. Adds a student to the program: adds n/John Doe p/98765432 tg/G17 sn/A1234567Z
  2. Execute help.
  3. Execute undo twice before succeeding in undoing the add command.
nus-se-script commented 1 week ago

Team's Response

We plan to implement this in the future. However, we believe this is not big enough of an issue since there is a feature to repeat the previous command by entering the up key which makes executing multiple undos very efficient.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]