Opened 7 months ago
Closed 6 months ago
#61884 closed defect (bug) (fixed)
Editor: Fix preloaded REST API paths
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.7 | Priority: | normal |
Severity: | normal | Version: | 6.0 |
Component: | Editor | Keywords: | has-patch |
Focuses: | Cc: |
Description
The two paths listed for preloading were silently failing.
See:
Change History (13)
This ticket was mentioned in PR #7179 on WordPress/wordpress-develop by @Mamaduka.
7 months ago
#1
- Keywords has-patch added
#2
@
7 months ago
When providing an incorrect path to preload, the rest_preload_api_request
will silently fail, and nothing will be preloaded.
- Fix typo for
wp_template_part
post type preload path for the Site Editor. - Do not preload the
wp_block
post type for post editors. The endpoint doesn't support unbound queries, and the data is no longer needed during editor initialization.
#3
@
7 months ago
- Owner set to Mamaduka
- Resolution set to fixed
- Status changed from new to closed
In 58904:
@Mamaduka commented on PR #7179:
7 months ago
#5
This ticket was mentioned in PR #7270 on WordPress/wordpress-develop by @Mamaduka.
7 months ago
#6
Permissions for creating pages (
OPTIONS /wp/v2/pages
). Both requests are made in theuseBlockEditorSettings
hook, at the top of the post editor React tree.
Matches the preloaded path for the posts editor and hopefully will improve the Site Editor loading time a bit.
Gutenberg Issue: https://github.com/WordPress/gutenberg/pull/64890
#7
@
7 months ago
- Resolution fixed deleted
- Status changed from closed to reopened
Re-opening as I found another issue with preload paths.
@Mamaduka commented on PR #7270:
7 months ago
#8
@noisysocks, I re-opened a similar ticket I had for 6.7.
@Mamaduka commented on PR #7270:
6 months ago
#9
@noisysocks, does this need another review/approval? It has been approved by @gziolo in the Gutenberg repo.
@noisysocks commented on PR #7270:
6 months ago
#10
You're a core committer so technically you can commit without review 😛 Best practice is to get a second pair of eyes on it though.
@Mamaduka commented on PR #7270:
6 months ago
#12
Committed in https://core.trac.wordpress.org/changeset/58984.
Gutenberg PR: https://github.com/WordPress/gutenberg/pull/64401 and https://github.com/WordPress/gutenberg/pull/64459.
Trac ticket: https://core.trac.wordpress.org/ticket/61884