Opened 2 years ago
Closed 2 years ago
#61884 closed defect (bug) (fixed)
Editor: Fix preloaded REST API paths
| Reported by: | Mamaduka | Owned by: | Mamaduka |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.7 |
| Component: | Editor | Version: | 6.0 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
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.
2 years ago
#1
- Keywords has-patch added
#2
@
2 years 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.
@Mamaduka commented on PR #7179:
2 years ago
#5
This ticket was mentioned in PR #7270 on WordPress/wordpress-develop by @Mamaduka.
2 years 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
@
2 years ago
- Resolution fixed
- Status closed → reopened
Re-opening as I found another issue with preload paths.
@Mamaduka commented on PR #7270:
2 years ago
#8
@noisysocks, I re-opened a similar ticket I had for 6.7.
@Mamaduka commented on PR #7270:
2 years ago
#9
@noisysocks, does this need another review/approval? It has been approved by @gziolo in the Gutenberg repo.
@noisysocks commented on PR #7270:
2 years 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:
2 years ago
#12
Committed in https://core.trac.wordpress.org/changeset/58984.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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