Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
PanelsIPELayoutForm wasn't updated for the core layout changes.
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#11 | 2858212-panels-11.patch | 4.42 KB | tim.plunkett |
#11 | 2858212-panels-11-interdiff.txt | 348 bytes | tim.plunkett |
#9 | 2858212-panels-9-interdiff.txt | 2.02 KB | tim.plunkett |
#9 | 2858212-panels-9.patch | 6.33 KB | tim.plunkett |
#5 | 2858212-panels-5.patch | 4.31 KB | tim.plunkett |
Comments
Comment #2
tim.plunkettComment #3
DamienMcKennaThis needs to be in 8.x-4.0-beta1.
Comment #5
tim.plunkettChanging the composer.json to make sure that's not the problem...
Comment #7
DamienMcKennaThe failures appear to be a problem with drupalci, specifically it's deleting modules after downloading them:
Comment #8
DamienMcKennaI opened up a testbot issue about this: #2858474: Dev dependencies are being removed before tests run?
Comment #9
tim.plunkettPanelizer's integration with Panels lives in panelizer, so should the test. This might help resolve a circular dependency.
Comment #11
tim.plunkettLet's talk about that interdiff in #2858488: Move PanelizerIntegrationTest from Panels to Panelizer, removing it from now since testbot won't notice the info.yml change in a patch anyway.
Trying a require-dev thing?
Comment #14
tim.plunkettPunting the rest to #2858552: Fix Panels composer.json, committing #2.