Skip to content

added patch for commerce kickstart install removing demo data#352

Open
dnotes wants to merge 1 commit into
omega8cc:masterfrom
dnotes:remove-demo-data-from-commerce-kickstart-install
Open

added patch for commerce kickstart install removing demo data#352
dnotes wants to merge 1 commit into
omega8cc:masterfrom
dnotes:remove-demo-data-from-commerce-kickstart-install

Conversation

@dnotes
Copy link
Copy Markdown
Contributor

@dnotes dnotes commented Aug 7, 2014

I've been reading about how to use Commerce Kickstart 2.x with BOA, and seems like it's a real hassle owing to the strange way that demo data is incorporated. I'm wondering if it would be possible to patch Commerce Kickstart so that the demo data isn't automatically installed? Am I correct that this patch should be automatically applied to the commerce_kickstart distribution in the Octopus platform? And that the old_short_name is no longer needed? If so, the change seems to allow new commerce kickstart sites to be installed without demo data. I got the idea from https://www.drupal.org/node/2191205.

Also if you could tell me how to test things like this, it would be great. First time trying to contribute to this project.

@omega8cc
Copy link
Copy Markdown
Owner

omega8cc commented Aug 8, 2014

We prefer to include the demo if the distro provides it. It is default behaviour also in many other distros.

While it may be inconvenient when you just want to use the distro for your needs and get rid of the demo, forcing the no-demo by default is not a correct approach.

Of course ideally it is the distro maintaners job to make it easy to remove the demo content without crashing the site and without forcing you to decide if you want the demo only at the install time.

If we will figure out how to do that, for example via some clever control file (the INI files wouldn't help in this case), we will add it as an optional feature in BOA, instead of forcing this for everyone.

Could you open this as a feature request instead?

Thanks!

@omega8cc
Copy link
Copy Markdown
Owner

We should revisit this, perhaps. Maybe as an option?

@omega8cc omega8cc reopened this Oct 22, 2015
@omega8cc omega8cc modified the milestones: 2.5.0, 3.x.0 Nov 14, 2015
@omega8cc
Copy link
Copy Markdown
Owner

omega8cc commented May 8, 2023

Better late than even later.. if anyone is still interested, we can reroll the patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants