Opened 15 months ago
Closed 14 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.
15 months ago
#1
- Keywords has-patch added
#2
@
15 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_partpost type preload path for the Site Editor. - Do not preload the
wp_blockpost type for post editors. The endpoint doesn't support unbound queries, and the data is no longer needed during editor initialization.
#3
@
15 months ago
- Owner set to Mamaduka
- Resolution set to fixed
- Status changed from new to closed
In 58904:
@Mamaduka commented on PR #7179:
15 months ago
#5
This ticket was mentioned in PR #7270 on WordPress/wordpress-develop by @Mamaduka.
14 months ago
#6
Permissions for creating pages (
OPTIONS /wp/v2/pages). Both requests are made in theuseBlockEditorSettingshook, 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
@
14 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:
14 months ago
#8
@noisysocks, I re-opened a similar ticket I had for 6.7.
@Mamaduka commented on PR #7270:
14 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:
14 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:
14 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