-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
ApiFetch: Script module #60952
base: trunk
Are you sure you want to change the base?
ApiFetch: Script module #60952
Conversation
c017fd8
to
9c47e04
Compare
Size Change: -36.5 kB (-2.05%) Total Size: 1.75 MB
ℹ️ View Unchanged
|
b509d08
to
2a33146
Compare
2a33146
to
7447563
Compare
lib/experimental/script-modules.php
Outdated
wp_register_script_module( '@wordpress/api-fetch', gutenberg_url( 'build/script-modules/apiFetch.min.js' ) ); | ||
|
||
// add_filter( | ||
// 'scriptmoduleconfig_@wordpress/api-fetch', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be restored with the correct filter name:
gutenberg/lib/experimental/script-modules.php
Line 246 in 5738e9b
$data = apply_filters( "script_module_data_{$module_id}", array() ); |
packages/api-fetch/src/wp-module.js
Outdated
|
||
if ( typeof document !== 'undefined' ) { | ||
const el = document.getElementById( | ||
'wp-scriptmodule-data_@wordpress/api-fetch' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be updated with the correct ID:
gutenberg/lib/experimental/script-modules.php
Line 284 in 5738e9b
'id' => "wp-script-module-data-{$module_id}", |
7447563
to
385ebb6
Compare
Yay |
What?
Why?
How?
Testing Instructions
Testing Instructions for Keyboard
Screenshots or screencast