WordPress.org
Get WordPress

Make WordPress Core

Opened 21 months ago

Closed 21 months ago

Last modified 21 months ago

#62465 closed defect (bug) (fixed)

Remove redundant server state from Interactivity Router

Reported by: czapla Owned by: desrosj
Priority: normal Milestone: 6.7.1
Component: Interactivity API Version: 6.7
Severity: normal Keywords: has-patch commit fixed-major dev-reviewed
Cc: Focuses:

Description

Remove the workaround for a bug that was fixed in ​https://github.com/WordPress/gutenberg/pull/66183.

Previously, if the store was not initialized with a minimal navigation object, the interactivity-router script module would error.

Change History (8)

#2 @czapla
21 months ago

  • Resolution → fixed
  • Status new → closed

Committed in [59416].

This ticket was mentioned in ​Slack in #core by desrosj. ​View the logs.
21 months ago

#5 @desrosj
21 months ago

  • Keywords fixed-major dev-feedback added
  • Resolution fixed
  • Status closed → reopened
  • Summary Interactivity API: Remove redundant server state from Interactivity Router → Remove redundant server state from Interactivity Router

Reopening to consider backporting.

#6 @desrosj
21 months ago

  • Keywords dev-reviewed added; dev-feedback removed
  • Version trunk → 6.7

[59416] looks good to backport.

#7 @desrosj
21 months ago

  • Owner set to desrosj
  • Resolution → fixed
  • Status reopened → closed

In 59436:

Interactivity API: Remove redundant server state from Interactivity Router.

Remove the workaround for a bug that was fixed in ​https://github.com/WordPress/gutenberg/pull/66183. Previously, if the store was not initialized with a minimal navigation object, the interactivity-router script module would error.

Reviewed by desrosj.
Merges [59416] to the 6.7 branch.

Props jonsurrell, czapla, gziolo.
Fixes #62465.

#8 @desrosj
21 months ago

  • Type enhancement → defect (bug)

It seems the other code changes relevant to this were shipped in 6.7, and this was overlooked /left out. Reclassifying as a bug to account for that.

Note: See TracTickets for help on using tickets.