Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Closed
WunderBart opened this issue Jun 26, 2020 · 3 comments
Closed

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

WunderBart opened this issue Jun 26, 2020 · 3 comments
Labels
[Goal] Gutenberg Working towards full integration with Gutenberg [Pri] BLOCKER Requires immediate attention. [Type] Bug When a feature is broken and / or not performing as intended

Comments

@WunderBart
Copy link
Member

WunderBart commented Jun 26, 2020 •
edited
Loading

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: #43676, #43525
@WunderBart WunderBart added [Type] Bug When a feature is broken and / or not performing as intended [Pri] BLOCKER Requires immediate attention. [Goal] Gutenberg Working towards full integration with Gutenberg labels Jun 26, 2020
@WunderBart WunderBart mentioned this issue Jun 26, 2020
Closed
19 tasks
@getdave
Copy link
Contributor

getdave commented Jun 29, 2020

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 sirreal mentioned this issue Jul 1, 2020
Closed
@sirreal
Copy link
Member

sirreal commented Jul 1, 2020 •
edited
Loading

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

@sirreal
Copy link
Member

sirreal commented Jul 3, 2020

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

@sirreal sirreal closed this as completed Jul 3, 2020
@fullofcaffeine fullofcaffeine mentioned this issue Jul 10, 2020
Closed
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[Goal] Gutenberg Working towards full integration with Gutenberg [Pri] BLOCKER Requires immediate attention. [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@getdave @sirreal @WunderBart