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

Process template part shortcodes before blocks (#50801) #50803

Merged
merged 1 commit into from May 20, 2023
Merged

Conversation

desrosj
Copy link
Contributor

@desrosj desrosj commented May 20, 2023

Process shortcodes before processing blocks so that dynamic blocks, by default, do not have shortcodes expanded

What?

Why?

How?

Testing Instructions

Testing Instructions for Keyboard

Screenshots or screencast

* Process shortcodes before processing blocks so that dynamic blocks, by default, do not have shortcodes expanded

* Revert "Process shortcodes before processing blocks so that dynamic blocks, by default, do not have shortcodes expanded"

This reverts commit 00374e0.

* Process shortcodes before processing blocks so that dynamic blocks, by default, do not have shortcodes expanded
@desrosj desrosj requested a review from ajitbohra as a code owner May 20, 2023 01:21
Copy link
Contributor

@antpb antpb left a comment

Choose a reason for hiding this comment

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

Working with WordPress 6.1.1 as expected on later versions.

@desrosj desrosj merged commit e7cf7ca into wp/6.1 May 20, 2023
19 of 23 checks passed
@desrosj desrosj deleted the 6-1-shortcodes branch May 20, 2023 01:59
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.

None yet

3 participants