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

Optimize preload paths for post and site editors #2387

Closed
wants to merge 1 commit into from

Conversation

jsnajdr
Copy link
Member

@jsnajdr jsnajdr commented Mar 8, 2022

This patch optimizes preload paths in post and site editor so that they match the real requests:

  • remove the / preload as the payload is very big and the response is not needed on any critical path
  • modify the preloaded path for /wp/v2/taxonomies so that it corresponds to what loadTaxonomyEntities requests. After Gutenberg version that ships the Load the entities list using the view context gutenberg#37685 patch is merged to core, these preloads will need to be further modified to use context=view instead of context=edit.
  • modify the /wp/v2/users/me path so that it matches the real request (no query params)
  • add a preload of /wp/v2/settings because it's requested on critical path (editor boot). Site editor already preloads this, we're just adding it to the post editor.

This is companion to Gutenberg PR WordPress/gutenberg#39256 which introduces compat code to modify the preload paths with a filter, when a Gutenberg plugin is active.

Fixes https://core.trac.wordpress.org/ticket/55337

@audrasjb
Copy link
Contributor

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.

2 participants