Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Loads of development has been going on in Fieldable Panels Panes (FPP), including patches that we've contributed. But we're still working off the current stable releases plus patches - it'd be great to try out the latest -dev code and help push it to release!
Issues to fix
Empty revision fieldset on existing FPPs. It doesn't look terribly useful - we should probably hide it:
Comment | File | Size | Author |
---|---|---|---|
#9 | panopoly_magic-update-fpp-2483265-9.patch | 755 bytes | dsnopek |
#9 | panopoly_test-update-fpp-2483265-9.patch | 892 bytes | dsnopek |
#9 | panopoly_core-update-fpp-2483265-9.patch | 1.42 KB | dsnopek |
Comments
Comment #1
dsnopekAttached is a patch to update to the latest -dev as of today! I'm going to run it through Travis in a moment.
EDIT: Here's the build on Travis: https://travis-ci.org/panopoly/panopoly/builds/61310253
Comment #2
dsnopekMy manual testing has been going really, really well!
I've only found one issue so far: There's an empty revision fieldset on existing FPPs. It doesn't look terribly useful - we should probably hide it:
Comment #3
dsnopekHere's a panopoly_magic patch that'll hide the revision fieldset.
Comment #4
mglamanThe following can be closed once this is committed: #2247353: Patch FPP to allow proper UUID subtype on page variant and mini panel export.
Comment #5
HaiNguyen007 CreditAttribution: HaiNguyen007 commentedI combine both patch in Update to latest Panelizer & Update to latest Fieldable Panels Panes (FPP) for panopoly-1.21 (for personal use & maybe benefit others); I'm tested all things work well so far.
Please see 3 patches in: 3 patches for updated panopoly 1.21 to use latest panelizer & fieldable panels panes
Comment #6
dsnopekThe last test build on Travis failed. The cause was changes in the way the edit form worked per #1911176: Cleanup edit form that our tests need to be updated for: basically, the "Make title a link" checkbox won't appear until there is a title, and we were clicking it before setting the title.
Attached is a patch that should fix that failure!
I'll run on Travis again in a moment...
EDIT: Here's the build on Travis-CI: https://travis-ci.org/panopoly/panopoly/builds/63231932
Comment #7
dsnopekA new commit was made to FPP, so uploading a new patch so we can run the tests on Travis again.
EDIT: Here is the Travis build: https://travis-ci.org/panopoly/panopoly/builds/63711788
Comment #8
dsnopekUpdating patch again for the latest commits. I'll run on Travis again in a moment!
EDIT: Here's the build on Travis: https://travis-ci.org/panopoly/panopoly/builds/64000165
Comment #9
dsnopekFPP 1.6 was just released:
https://www.drupal.org/node/2497261
Here are re-rolls of all the patches here, which I'm going to run through Travis one last time.
EDIT: Here's the Travis build: https://travis-ci.org/panopoly/panopoly/builds/64608448
Comment #10
dsnopekTested passed on Travis! Committed. :-)