Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added patch for commerce kickstart install removing demo data #352

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dnotes
Copy link
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
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
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
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.

None yet

2 participants