#58230 closed enhancement (fixed)
Disable lazy loading term meta get_block_templates.
Reported by: | spacedmonkey | Owned by: | spacedmonkey |
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Editor | Keywords: | gutenberg-merge has-patch |
Focuses: | performance | Cc: |
Description
Backport from https://github.com/WordPress/gutenberg/pull/47999
There is no need to lazy load in get_block_templates
Change History (8)
This ticket was mentioned in Slack in #core-editor by spacedmonkey. View the logs.
18 months ago
This ticket was mentioned in PR #4411 on WordPress/wordpress-develop by @spacedmonkey.
18 months ago
#2
- Keywords has-patch added
@oandregal commented on PR #4411:
18 months ago
#4
Added Andrei as reviewer. If the original reviewers can be tagged in wordpress-develop, I suggest adding them here, as they know best about the changes. It also helps the review load to be distributed among everyone.
Code-wise the changes are the same from the original PR. There are no test instructions in any place I've checked (here, trac ticket, or the original PR), so I don't know what I should test or what I need to look at. I'll leave space for others to comment here (happy to help if test instructions are provided).
@spacedmonkey commented on PR #4411:
17 months ago
#5
@oandregal As this is a performance improvement, there is no functional change. So that is not to test other than if the code make sense.
#6
@
17 months ago
- Owner set to spacedmonkey
- Resolution set to fixed
- Status changed from new to closed
In 55817:
Trac ticket: https://core.trac.wordpress.org/ticket/58230#ticket