Backport Changelog: Link Core PR #12007 for #75793#78786
Backport Changelog: Link Core PR #12007 for #75793#78786adamsilverstein wants to merge 2 commits into
Conversation
Adds the missing 7.1 backport-changelog entry pairing the Core backport PR WordPress/wordpress-develop#12007 with the original Gutenberg PR #75793 (Media: Move image output format filtering to upload response). #75793 was merged with the legacy `No Core Sync Required` label; the Core sync is now required since client-side media processing was re-introduced in Core.
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
t-hamano
left a comment
t-hamano
left a comment
There was a problem hiding this comment.
I believe the backport changelog should reflect the context in which the PR was originally merged. In other words, shouldn't the backport-changelog/7.0 directory be the correct one?
t-hamano
added
the
[Type] Task
oh, i didn't understand that! @t-hamano - to clarify, the feature will only land in the 7.1 cycle, but the merge was made during the 7.0 cycle. So that means the backport link goes in the 7.0 folder? Is this documented somewhere? |
t-hamano
left a comment
t-hamano
left a comment
There was a problem hiding this comment.
to clarify, the feature will only land in the 7.1 cycle, but the merge was made during the 7.0 cycle. So that means the backport link goes in the 7.0 folder?
I think so. I believe this backport changelog should have been added when it was merged, so we should respect the cycle that was intended at that time.
P.S. Since the client-side media feature was not released in 7.0, should this file be moved to the 7.1 directory?
2 participants
Summary
Adds the missing 7.1 backport-changelog entry pairing the Core backport PR WordPress/wordpress-develop#12007 with the original Gutenberg PR #75793 (Media: Move image output format filtering to upload response).
#75793 was merged with the legacy
No Core Sync Requiredlabel. Since client-side media processing has been re-introduced in Core (https://core.trac.wordpress.org/changeset/62428), the Core sync is now required. Tracked at https://core.trac.wordpress.org/ticket/65367.Test plan
backport-changelog/7.1/12007.mdcontent matches the format documented inbackport-changelog/readme.md.