Navigation: Streamline preloading requests #51803
Assignees
Labels
[Block] Navigation
Affects the Navigation Block
Comments
scruffian
moved this to New issues
in Ignite board - ARCHIVED - USE https://github.com/orgs/WordPress/projects/122
scruffian
moved this from New issues
to Todo
in Ignite board - ARCHIVED - USE https://github.com/orgs/WordPress/projects/122
scruffian
moved this from Todo
to In Progress
in Automattic team Ignite's project board
github-project-automation
bot
moved this from In Progress
to Done
in Automattic team Ignite's project board
github-project-automation
bot
moved this from Todo
to Done
in Ignite board - ARCHIVED - USE https://github.com/orgs/WordPress/projects/122
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What problem does this address?
We preload two different URLs for navigation, but they are almost identical: https://github.com/WordPress/gutenberg/blob/trunk/lib/compat/wordpress-6.3/navigation-block-preloading.php#L31
We should find a way to streamline this so that we only need to load one.
The difference between the two is the __locale parameter. This is necessary because in the sidebar we load navigation menus using
useEntityRecords, but in the block we usegetEntityRecords. We should probably switch to usinguseEntityRecordsin both places.The text was updated successfully, but these errors were encountered: