nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Multiple instances of undo are allowed. #1502

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Summary: When undo is typed in multiple times, it only undo-s one command

Steps to reproduce:

  1. add n/seannnnnnnnnnnnnnnnnnn c/82230350 e/sean@gmail.coms
  2. undo undo undo undo

Expected results: Either 1a. Four Commands to be undone. 1b. No Commands to be undone.

Actual results: Only one command is undone.

Screenshot of issue: image.png


[original: nus-cs2103-AY1920S1/pe-interim#1511]

zhixianggg commented 4 years ago

Team's Response

According to UG,

5.7.1. Undo: undo

Undo the previous command.

Format: undo

We did not allow undo to be keyed in consecutively. In our UG, we clearly stated that the proper command to be executed is undo, not (undo)*, with * denoting n number of undoable commands.

Duplicate status (if any):

--