Project

General

Profile

Bug #7775

Uploading new CSS style requires a composer install

Added by Yannick Warnier over 4 years ago. Updated over 4 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Styles, Layout & Accessibility
Target version:
Start date:
15/07/2015
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

When uploading a new CSS style through the platform settings configuration page (example attached), it is necessary to launch a "composer install" command manually to be able to see the style.

Could we find a way to avoid this? (for example copying the style to web/ when uploaded through the web form?)


Files

cancun.zip (175 KB) cancun.zip Yannick Warnier, 15/07/2015 07:42

Associated revisions

Revision c305b376 (diff)
Added by Julio Montoya over 4 years ago

Fix theme upload. see #7775

History

#1

Updated by Yannick Warnier over 4 years ago

  • Description updated (diff)
#3

Updated by Julio Montoya over 4 years ago

  • Assignee changed from Julio Montoya to Yannick Warnier

This will require that the web/css/themes folder will be writable in the installation process.

#4

Updated by Yannick Warnier over 4 years ago

  • Assignee changed from Yannick Warnier to Julio Montoya

Or that we check the permissions before offering the possibility to upload, indicating in the upload page that this folder must be writeable.
Although I would prefer that it be required during installation for now (otherwise it's a bit confusing for users).

OK, you have my agreement to set it as a requirement (even if it goes against our simplification :-))

#5

Updated by Julio Montoya over 4 years ago

  • Status changed from Assigned to Needs more info

I just applied a fix.

1. The web folder must be writeable (asked in the installation process)
2. When upload a new css theme the folder is copied here:

app/Resources/public/css/themes

and then mirrored here: web/css/themes/cancun

So we can still delete web/* and then execute composer update, the files will be auto generate.

#6

Updated by Yannick Warnier over 4 years ago

  • Status changed from Needs more info to Bug resolved
  • % Done changed from 0 to 100

Perfect. Long live the Montoyator!

Also available in: Atom PDF