Panopoly core is currently using Entity 1.6 along with several patches. This is several versions (and 2 years) out of date and there have been many useful fixes, although sadly none of the patches from panopoly_core have been committed still.

This issue is to talk about updating to the latest release of Entity.

CommentFileSizeAuthor
#2 panopoly_core-update-entity-2867870-2.patch397 bytescboyden
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mpotter created an issue. See original summary.

cboyden’s picture

cboyden CreditAttribution: cboyden commented
Status: Active » Needs review
FileSize
panopoly_core-update-entity-2867870-2.patch397 bytes

Here's a patch to start testing.

EDIT: Travis build at https://travis-ci.org/cboyden/panopoly/builds/223628628

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented

Thanks!

The issue description says that we have some patches, but it looks like we don't... which is a good thing! :-)

The last Travis build failed because of #2869464: All Chrome tests failing with "Could not open connection: An unknown server-side error" so here is a new one:

https://travis-ci.org/panopoly/panopoly/builds/232595699

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented

Here's a new Travis build after some attempts to fix the tests:

https://travis-ci.org/panopoly/panopoly/builds/235003321

  • dsnopek committed bca377d on 7.x-1.x 
    Update Panopoly Core for Issue #2867870 by cboyden: Update Entity API to...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Needs review » Fixed

The tests pass, and this looks good to me. Thanks, Everyone!

  • dsnopek committed bca377d on 8.x-2.x 
    Update Panopoly Core for Issue #2867870 by cboyden: Update Entity API to...

Status: Fixed » Closed (fixed)

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