Problem/Motivation

In #2624874: Provide a render element we added a form element for entity browser. At the same time APIs that were used to display entity browser before were changed, which means that the Entity browser integration in Entity embed is currently broken.

Proposed resolution

Update Entity browser integration to use the provided form element.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

slashrsm created an issue. See original summary.

slashrsm’s picture

Status: Active » Needs review
FileSize
1.93 KB

Status: Needs review » Needs work

The last submitted patch, 2: 2767573_2.patch, failed testing.

slashrsm’s picture

Status: Needs work » Needs review

Expected fail. Changes are not in EB's tagged release yet and tests are running on old code as a result. We should sync releases of all modules that are affected.

slashrsm’s picture

Passes locally.

jlandfried’s picture

Status: Needs review » Reviewed & tested by the community

Thanks! This fixed my broken embed buttons, using the latest dev releases of entity_embed and entity_browser. Can't speak to syncing the releases, but this got me up and running again.

samuel.mortenson’s picture

This is working for me as well.

slashrsm’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

  • slashrsm committed b5c62bb on 8.x-1.x
    Issue #2767573 by slashrsm, jiff, samuel.mortenson: Use Entity browser's...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.