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

[PE-D] Undo undoes beyond done actions #134

Closed
nus-pe-script opened this issue Oct 30, 2021 · 0 comments · Fixed by #179
Closed

[PE-D] Undo undoes beyond done actions #134

nus-pe-script opened this issue Oct 30, 2021 · 0 comments · Fixed by #179
Assignees
Milestone

Comments

@nus-pe-script
Copy link

It seems like I can undo things that I did not even do in the first place. The following are the steps I took:

Start:

image.png

Undo, (correctly does not undo as expected):

image.png

After undoable command,
add n/Jane Doherty p/1847291074 e/janedoe123@gmail.com a/Coral Street, 5th Avenue, #02-04, pr/500.00 i/none s/pending t/caterer:
image.png

Undo, (correctly undoes as expected):

image.png

Undo again, (erroneously undoes unexpectedly):

image.png

Undo all the way, (still undoes unexpectedly until all are deleted even when I didn't add them):

image.png


Labels: severity.Medium type.FunctionalityBug
original: jayasting98/ped#4

@Banana3021 Banana3021 linked a pull request Nov 3, 2021 that will close this issue
@Banana3021 Banana3021 added this to the v1.4 milestone Nov 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants