? Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
31 Jan 2014

This modifies the 3.x build script to exclude the sample images we distribute from the update packages. They serve no real purpose once a site is established and site owners who wish to remove this content should be free to do so. Note that the full install package is unaffected and these images will continue to be bundled there.

I've done this as a PR to solicit feedback in case there are individuals who may think these are important to retain in the update packages.

avatar mbabker mbabker - open - 31 Jan 2014
avatar brianteeman
brianteeman - comment - 31 Jan 2014

Yes please - its very frustrating to remove them only for them to come back again.

avatar drmmr763
drmmr763 - comment - 31 Jan 2014

+1. Also I'm curious what happens if I've deleted a core-image and uploaded my own under the same name, and then run the updater. Does my image get overwritten?

avatar mbabker
mbabker - comment - 31 Jan 2014

Presumably, yes, your image would get overwritten.

On Thu, Jan 30, 2014 at 6:55 PM, Chad Windnagle notifications@github.comwrote:

+1. Also I'm curious what happens if I've deleted a core-image and
uploaded my own under the same name, and then run the updater. Does my
image get overwritten?

Reply to this email directly or view it on GitHub#3013 (comment)
.

avatar drmmr763
drmmr763 - comment - 31 Jan 2014

Then that's definitely a reason to chuck 'em. Could result in overwriting my content which should never happen.

avatar karantzis
karantzis - comment - 31 Jan 2014

Please remove. Helps keep things uncluttered a bit more.

avatar Bakual
Bakual - comment - 31 Jan 2014

Great change! :+1:

avatar toretto
toretto - comment - 31 Jan 2014

Get rid of them. It's already bad enough they're there when you never want to install the sample data to begin with. ;-)

avatar mbabker
mbabker - comment - 31 Jan 2014

I'll merge this in when I have a few minutes to do the same for 2.5 and the nightly build scripts.

avatar mbabker mbabker - reference | 57ecb1d - 3 Feb 14
avatar mbabker mbabker - merge - 3 Feb 2014
avatar mbabker mbabker - close - 3 Feb 2014
avatar mbabker mbabker - change - 3 Feb 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-02-03 19:05:29
avatar mbabker mbabker - close - 3 Feb 2014
avatar mbabker mbabker - head_ref_deleted - 3 Feb 2014
avatar mbabker mbabker - reference | deb98fe - 3 Feb 14
avatar Bakual Bakual - reference | 4b657cd - 12 May 14

Add a Comment

Login with GitHub to post a comment