Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add undo feature #151

Merged

Conversation

hanstirtaputra
Copy link

Undo feature reverts the state of the model to the last changed state. Undo command does not undo commands that does not change the state of the ModelManager, for example listsvc or findsvc.

Hans Sebastian Tirtaputra added 6 commits October 15, 2020 14:01
Fix issue where history manager stores model states that are identical. This issue arose when the user enters commands that does not mutate the model
state such as list and find commands.
@hanstirtaputra hanstirtaputra added this to the v1.3 milestone Oct 15, 2020
@hanstirtaputra hanstirtaputra self-assigned this Oct 15, 2020
@hanstirtaputra hanstirtaputra added priority.Medium Nice to have and removed priority.High Must do labels Oct 15, 2020
Copy link

@RuiFengg RuiFengg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but can consider changing the message to user 😸

@galvinleow galvinleow merged commit 37124e4 into AY2021S1-CS2103T-W13-3:master Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.Medium Nice to have type.Story A user story
Projects
None yet
Development

Successfully merging this pull request may close these issues.

As a home-based beauty salon owner, I want to undo my previous command
3 participants