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

Script Modules: Adjust filter and ID #62170

Merged
merged 2 commits into from
Jun 4, 2024
Merged

Conversation

sirreal
Copy link
Member

@sirreal sirreal commented May 31, 2024

What?

Update the Script Module data filter and script tag ID to align with WordPress/wordpress-develop#6682.

This feature was just implemented in Gutenberg and not yet released, but Core feedback has suggested some changes to the filter name and script tag ID used.

Why?

If the initial version uses the filter and script tag IDs that will ultimately be used in Core, it should save work in the future.

Copy link

This pull request has changed or added PHP files. Please confirm whether these changes need to be synced to WordPress Core, and therefore featured in the next release of WordPress.

If so, it is recommended to create a new Trac ticket and submit a pull request to the WordPress Core Github repository soon after this pull request is merged.

If you're unsure, you can always ask for help in the #core-editor channel in WordPress Slack.

Thank you! ❤️

View changed files
❔ lib/experimental/script-modules.php

@sirreal sirreal force-pushed the update/script-module-data-tweaks branch from 4b4db3a to d8c0e58 Compare June 3, 2024 16:23
@sirreal sirreal marked this pull request as ready for review June 3, 2024 16:57
Copy link

github-actions bot commented Jun 3, 2024

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 props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: sirreal <jonsurrell@git.wordpress.org>
Co-authored-by: gziolo <gziolo@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@sirreal sirreal added Backport to Gutenberg RC Pull request that needs to be backported to a Gutenberg release candidate (RC) [Type] Task Issues or PRs that have been broken down into an individual action to take labels Jun 3, 2024
@sirreal sirreal requested a review from gziolo June 3, 2024 17:18
@sirreal sirreal merged commit bbf0816 into trunk Jun 4, 2024
66 of 72 checks passed
@sirreal sirreal deleted the update/script-module-data-tweaks branch June 4, 2024 05:41
@github-actions github-actions bot added this to the Gutenberg 18.6 milestone Jun 4, 2024
@sirreal sirreal added the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Jun 4, 2024
carstingaxion pushed a commit to carstingaxion/gutenberg that referenced this pull request Jun 4, 2024
…62170)

Update the Script Module data filter name and script tag ID to align with Core changes.

See WordPress/wordpress-develop#6682.

Co-authored-by: sirreal <jonsurrell@git.wordpress.org>
Co-authored-by: gziolo <gziolo@git.wordpress.org>
@sirreal sirreal modified the milestones: Gutenberg 18.6, Gutenberg 18.5 Jun 5, 2024
vcanales pushed a commit that referenced this pull request Jun 5, 2024
Update the Script Module data filter name and script tag ID to align with Core changes.

See WordPress/wordpress-develop#6682.

Co-authored-by: sirreal <jonsurrell@git.wordpress.org>
Co-authored-by: gziolo <gziolo@git.wordpress.org>
@vcanales
Copy link
Member

vcanales commented Jun 5, 2024

I just cherry-picked this PR to the release/18.5 branch to get it included in the next release: 2fe4dfa

@vcanales vcanales removed the Backport to Gutenberg RC Pull request that needs to be backported to a Gutenberg release candidate (RC) label Jun 5, 2024
@sirreal
Copy link
Member Author

sirreal commented Jun 6, 2024

I added the Backport to WP Beta/RC label here. The only backport necessary is the @wordpress/interactivity package that should be updated to the version released in the latest Gutenberg version 18.5.

ellatrix pushed a commit that referenced this pull request Jun 11, 2024
Update the Script Module data filter name and script tag ID to align with Core changes.

See WordPress/wordpress-develop#6682.

Co-authored-by: sirreal <jonsurrell@git.wordpress.org>
Co-authored-by: gziolo <gziolo@git.wordpress.org>
ellatrix pushed a commit that referenced this pull request Jun 11, 2024
Update the Script Module data filter name and script tag ID to align with Core changes.

See WordPress/wordpress-develop#6682.

Co-authored-by: sirreal <jonsurrell@git.wordpress.org>
Co-authored-by: gziolo <gziolo@git.wordpress.org>
patil-vipul pushed a commit to patil-vipul/gutenberg that referenced this pull request Jun 17, 2024
…62170)

Update the Script Module data filter name and script tag ID to align with Core changes.

See WordPress/wordpress-develop#6682.

Co-authored-by: sirreal <jonsurrell@git.wordpress.org>
Co-authored-by: gziolo <gziolo@git.wordpress.org>
@ellatrix ellatrix removed the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Jun 18, 2024
@ellatrix
Copy link
Member

This was cherry-picked to the wp/6.6 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants