Skip to content

Update the script registration of the react scripts with a cache version#46768

Merged
youknowriad merged 1 commit into
trunkfrom
fix/react-browser-cache-issue
Dec 23, 2022
Merged

Update the script registration of the react scripts with a cache version#46768
youknowriad merged 1 commit into
trunkfrom
fix/react-browser-cache-issue

Conversation

@youknowriad

Copy link
Copy Markdown
Contributor

closes #46737

What?

We did an update to React in the previous Gutenberg release 14.8 but it seems the script url didn't change, the version that is used for the script is the WP version which remains stable between two gutenberg versions. This PR forces the use of the React version for this script as a short term fix.

Long term, we should try to use the version from the package.json file somehow.

Testing Instructions

1- Clear browse cache
2- Use a Gutenberg version prior to 14.7 (or 14.7)
3- Load the editor
4- Update the Gutenberg version to 14.8 (or this PR)
5- If you try to save the post, you'll have an error in 14.8 but it should work with this PR.

@youknowriad youknowriad added the [Type] Bug An existing feature does not function as intended label Dec 23, 2022
@youknowriad youknowriad added this to the Gutenberg 14.8 milestone Dec 23, 2022
@youknowriad youknowriad self-assigned this Dec 23, 2022
@youknowriad youknowriad merged commit b95339b into trunk Dec 23, 2022
@youknowriad youknowriad deleted the fix/react-browser-cache-issue branch December 23, 2022 15:28
ramonjd added a commit that referenced this pull request Mar 25, 2026
- Extend wp-build generated constants.php with react_version from root
  package.json (devDependencies or dependencies react field).
- Define GUTENBERG_REACT_VENDOR_VERSION in load.php when present.
- Pass that version to react, react-dom, and react-jsx-runtime registration;
  fall back to major "18" for older builds or projects without a react pin.

Follow-up to the approach in #46768, tying ?ver= to the actual bundled React.

Made-with: Cursor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TypeError: (0 , b.useId) is not a function

2 participants