-
Notifications
You must be signed in to change notification settings - Fork 179
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
Autosaves: delete local auto-draft autosave after save #12417
Conversation
Plugin builds for 8411470 are ready 🛎️!
|
Size Change: +45 B (0%) Total Size: 2.72 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to work great!
packages/story-editor/src/components/localAutoSaveHandler/localAutoSave.js
Outdated
Show resolved
Hide resolved
@miina @swissspidy - When I loaded the So I cannot test this ticket with the changes in Q: Could you help me get a better build made without this problem ? |
Please try again now. The PR is updated now and should not have this bug anymore. |
@miina @swissspidy - I can confirm that the So the sequence I observed was:
NOW the
I was expecting the Q: Could you confirm this sequence is ok ? |
In addition, I would like to flag a regression: When the Steps
ISSUE: A blank story is created This is working as expected on Cc: @miina |
@kkalarickal Yes, that's ok, thanks for checking. Since the saved version is also stored in the database then there's no need for an older local backup. |
Confirmed the issue and should be fixed now as well, could you test again? |
Confirmed that the There are some more scenarios around take over and auto-saved session objects ... but those are unrelated to this ticket. I will file them separately. @miina @swissspidy - this ticket can be merged. |
Context
Summary
Relevant Technical Choices
To-do
User-facing changes
Testing Instructions
This PR can be tested by following these steps:
wp_stories_autosave_story_auto-draft
gets createdwp_stories_autosave_story_12345
gets createdwp_stories_autosave_story_auto-draft
is discarded at this pointReviews
Does this PR have a security-related impact?
Does this PR change what data or activity we track or use?
Does this PR have a legal-related impact?
Checklist
Type: XYZ
label to the PRFixes #12342