This is a sort of meta issue to cover the things needed to switch to using Migrate for demo content. Most of this is implemented in the following three patches.

#2105433: Add Migrate modules to Panopoly Core to help facilitate better demo content patterns
#2105427: Implement Migrate module for Panopoly News demo content
#1869972: Implement Migrate Module for Panopoly Demo Content
#2105513: Implement Migrate module for Panopoly FAQ demo content

Much of the original discussion around this took place in #1869972: Implement Migrate Module for Panopoly Demo Content where several options were discussed, the main 2 contenders being UUID Features and Migrate. There was a general consensus that Migrate was the best way forwards.

CommentFileSizeAuthor
#1 2105503-panopoly-migrate.patch2.06 KBmrfelton

Comments

mrfelton’s picture

Issue summary: View changes

Fixing issue links

mrfelton’s picture

Issue summary: View changes

Add reference to panopoly_faq issue

mrfelton’s picture

Issue summary: View changes

Add some reasoning

mrfelton’s picture

Status: Active » Needs review
StatusFileSize
new2.06 KB

To support this, a change is needed to the install profile to make it aware that demo content may be coming from multiple apps. Since each app provides its demo content within a submodule, the Apps install task needs to be made aware of this.

This patch also re-enables the 'Install demo content' checkbox in the install profile as this is needed in order to let the user decide wether they want all of the demo content installed or not. The previous logic assumed that the panopoly_demo module was the only module providing default content. Actually, we have Panopoly Demo, Panopoly News and Panopoly FAQ that all provide their own demo content.

mrfelton’s picture

Issue summary: View changes

.

mrfelton’s picture

Version: 7.x-1.x-dev » 7.x-1.0-rc5
Assigned: Unassigned » mrfelton
Issue summary: View changes
lsolesen’s picture

What about extending the demo behat test to test what needs to be tested to be able to commit this? @mrfelton Do you have a list of what should be tested?

mrfelton’s picture

Well, the test would be to check that the various pieces of demo content are created properly when the demo module is enabled, and removed again when its disabled. I can provide a more comprehensive list.

But also I feel that an important part of these patches that need to be tested before they can be committed, which I have not yet tested at all is what happens when a site on RC5 or earlier with demo content enabled is upgraded to the latest code. Does the demo content just disappear? is it properly recreated? etc. I haven't tested that at all yet. Not sure if the behat setup can help there though since its only set up to test new installs at this pointI think.

lsolesen’s picture

mrfelton’s picture

Status: Needs review » Fixed

This was actually fixed before the 1.1 release. Seems this ticket wasn't closed out properly.

Status: Fixed » Closed (fixed)

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