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

Make sure DocumentChange.timestamp is always set #4360

Merged
merged 1 commit into from Dec 10, 2019
Merged

Conversation

J-Fields
Copy link
Member

This seems like it could be incorrect sometimes, and we should make sure all changes are either declared finished at their creation or finished before they can be undone.
Also remove an instance of suite.only() that slipped in...whoops
Fixes #4351

This seems like it could be incorrect sometimes, and we should make sure all changes are either declared finished at their creation or finished before they can be undone.
Also remove an instance of suite.only() that slipped in...whoops
Fixes #4351
@J-Fields J-Fields merged commit 67a505b into master Dec 10, 2019
@J-Fields J-Fields deleted the fix-timestamp branch December 14, 2019 20:29
@raudabaugh
Copy link

When is the next release expected? Would love to start using this fix 👍

@J-Fields
Copy link
Member Author

@raudabaugh Sorry, was meaning to get a few other small things into the patch release but life got busy. I'll do my best to release tonight.

@changhz
Copy link

changhz commented Dec 24, 2019

this annoying error has been torturing me 😭😭

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 this pull request may close these issues.

Undo After Replace Causes Error: "Cannot read property 'getTime' of undefined."
3 participants