nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Undo list #2545

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Able to undo though previous function is list-s. The undone command is not list-s though.

E.g. add-s n/NTUC Fairprice Macpherson Mall c/63521728 e/MacphersonMall@NTUCFairprice.com a/401, #02-22 MacPherson Rd, Macpherson Mall, 368125 gp/banana 5 gp/tissue paper 0.55 list-s undo


[original: nus-cs2103-AY1920S2/pe-interim#2536]

NicholasCF commented 4 years ago

Team's Response

This is the expected behavior. undo will only undo commands that modify the data itself. Commands such as list-s are ignored in considering what to undo.

Here is the snippet in the UG:

Screen Shot 2020-04-18 at 9.19.07 PM.png

Duplicate status (if any):

--