? Success

User tests: Successful: Unsuccessful:

avatar smz
smz
19 Dec 2014

Two more...

avatar smz smz - open - 19 Dec 2014
avatar jissues-bot jissues-bot - change - 19 Dec 2014
Labels Added: ?
avatar infograf768
infograf768 - comment - 19 Dec 2014

Please add the 2 directories in script.php
see
#5467 (comment)

avatar smz
smz - comment - 19 Dec 2014

@infograf768 Done: in script.php I also added the files (i.e. index.html) and the directories for this and my previous similar PR #5467. Is that OK?

avatar Bakual
Bakual - comment - 19 Dec 2014

I don't think we need to remove the index.html files during an update. There is no harm done when they stay there and I honestly don't want to see all 1000 files listed in that method.

Is there a reason to add those files to the script?

avatar Bakual
Bakual - comment - 19 Dec 2014

Ah sorry, it's about whole folders which aren't needed. Then it's fine of course.

avatar sovainfo
sovainfo - comment - 19 Dec 2014

Checked template editor and it doesn't seem to have a problem with it.

avatar infograf768 infograf768 - change - 20 Dec 2014
Milestone Added:
avatar infograf768 infograf768 - reference | c37da36 - 20 Dec 14
avatar infograf768 infograf768 - merge - 20 Dec 2014
avatar infograf768 infograf768 - close - 20 Dec 2014
avatar infograf768 infograf768 - close - 20 Dec 2014
avatar infograf768 infograf768 - change - 20 Dec 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-12-20 06:58:21
avatar infograf768
infograf768 - comment - 20 Dec 2014

Thanks. Merged.

avatar smz smz - head_ref_deleted - 21 Dec 2014

Add a Comment

Login with GitHub to post a comment