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.

Sign up for GitHub

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

Jump to bottom

Bug fixes for WordPress 5.3 RC 3 #18131

Merged
merged 5 commits into from
Oct 28, 2019
Merged

Bug fixes for WordPress 5.3 RC 3 #18131

merged 5 commits into from
Oct 28, 2019

Conversation

youknowriad
Copy link
Contributor

Cherry-picked PRs to include in WordPress 5.3 RC3

#18108
#18051
#18106
#18102

mcsf and others added 4 commits October 28, 2019 11:23
@mcsf @youknowriad
Local autosave: Clear after successful save (#18051)
7421040 
* Local autosave: Clear after successful save

Presumably, somewhere in the fixing of conflicts between remote and
local autosaves (purge local upon successful remote autosave),
LocalAutosaveMonitor stopped purging the local autosave upon successful
*saves*.
@jasmussen @youknowriad
Fix checkboxes for postmeta. (#18108)
f9b1c20
@ellatrix @youknowriad
Preserve attributes on split (#18102)
1387889
@youknowriad
Allow media upload post processing for all 5xx responses (#18106)
107693f
@youknowriad youknowriad self-assigned this Oct 28, 2019
@youknowriad youknowriad requested a review from a team October 28, 2019 10:29
@youknowriad
Copy link
Contributor Author

We have the same test failures as the RC2 PR which means we can move forward.

mcsf
mcsf approved these changes Oct 28, 2019
View reviewed changes
Copy link
Contributor

@mcsf mcsf left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@youknowriad youknowriad merged commit 2a87ef7 into wp/5.3 Oct 28, 2019
@youknowriad youknowriad deleted the fix/bug-fixes-5.3-rc.3 branch October 28, 2019 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mcsf mcsf mcsf approved these changes

@ajitbohra ajitbohra Awaiting requested review from ajitbohra ajitbohra is a code owner

@jorgefilipecosta jorgefilipecosta Awaiting requested review from jorgefilipecosta jorgefilipecosta is a code owner

@mmtr mmtr Awaiting requested review from mmtr mmtr is a code owner

@nerrad nerrad Awaiting requested review from nerrad nerrad is a code owner

@Soean Soean Awaiting requested review from Soean Soean is a code owner

@talldan talldan Awaiting requested review from talldan talldan is a code owner

Assignees

@youknowriad youknowriad

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@youknowriad @mcsf @jasmussen @ellatrix