search

Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.42k stars 1.99k forks source link

Gutenberg v8.4.0: Navigation block broken for .com #43736

Closed WunderBart closed 4 years ago

WunderBart commented 4 years ago

The Navigation block is stuck in the placeholder state, throwing 404 for the __experimental/menus endpoint:

image

⚠️ This is wpcom specific and doesn't happen on AT sites - everything seems to be working well there.

Steps to reproduce

  1. If you're an Automattician, see p7DVsv-8Te-p2 for how to get v8.4.0 on your test site,
  2. Add a Navigation block to your site,
  3. See the spinner loading the nav placeholder menu forever.
  4. Open JS console to see a rest_no_route error.

Context / Source

Related: https://github.com/Automattic/wp-calypso/issues/43676, https://github.com/Automattic/wp-calypso/issues/43525

getdave commented 4 years ago

The Menus endpoint was added when we enabled the ability to create Navigation blocks from existing WP Menus. I think that Blocks are allowed to depend on experimental features so this is technically valid.

I think perhaps we might need to focus on allow Dotcom to support these experimental endpoints?

sirreal commented 4 years ago

Figuring out __experimental routes: p3hLNG-174-p2 Also: pMz3w-bDy-p2

sirreal commented 4 years ago

Fixed by pMz3w-bDy-p2 / r209932-wpcom.