nus-cs2103-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Undo does not work for assignment commands #1235

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago

Under the undo feature in the UG, it says that only scheduling commands are not supported, however, addassignment and deleteassignment commands are also not supported.

I tried to undo the addassignment command for PA1, but is unable to do so, it even skips the addassignment command and undos my addmodule command in the gif.

(The gif follows immediately from #6 No success message after adding assignment, where i added an assignment to CS3230)

a1I68fuG02.gif


[original: nus-cs2103-AY2021S1/pe-interim#1121]

murtubak commented 3 years ago

Team's Response

Thank you for the feedback. There was a documentation flaw in the UG which did not include assignment commands to not work with undo and redo. Like the scheduler commands, undo and redo commands have not been extended to work with the assignment commands.

As such, the severity of this issue should lie with the documentation flaw and has been dropped to medium as the user can still delete the assignment and add a new assignment if necessary and the undo function accidentally undoing something that the user did not want to undo can be reverted with redo.

Duplicate status (if any):

--