Skip to content

Conversation

@youknowriad
Copy link
Contributor

Fix #12147

Testing instructions

  • Publish a post
  • Edit the post
  • Wait for autosave
  • The "reverted to draft" message should not show up

@youknowriad youknowriad self-assigned this Nov 21, 2018
@youknowriad youknowriad requested review from a team and gziolo November 21, 2018 07:42
Copy link
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

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

I will test soonish.


// Autosaves are neither shown a notice nor redirected.
if ( isAutosave ) {
if ( get( action.options, [ 'isAutosave' ] ) ) {
Copy link
Member

Choose a reason for hiding this comment

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

Yes, we changed that. Now, I remember.

@gziolo gziolo added the [Type] Bug An existing feature does not function as intended label Nov 21, 2018
@gziolo gziolo added this to the 4.6 milestone Nov 21, 2018
@youknowriad youknowriad modified the milestones: 4.6, 4.5.1 Nov 21, 2018
@gziolo
Copy link
Member

gziolo commented Nov 21, 2018

Unit test failed

@youknowriad
Copy link
Contributor Author

yes, looking.

Copy link
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

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

I confirm it fixes the issue.

@youknowriad youknowriad merged commit e05eb6d into master Nov 21, 2018
@youknowriad youknowriad deleted the fix/autosave-revert-to-draft-message branch November 21, 2018 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants