Devfront: internalisation de la librairie semantic-ui#2179
Devfront: internalisation de la librairie semantic-ui#2179vgreb wants to merge 2 commits intoafup:masterfrom
Conversation
vgreb
changed the title
vgreb
changed the title
vgreb
force-pushed
the
feature/semantic-ui-no-cdn
branch
5 times, most recently
from
07b4e93 to
d3601e1
Compare
There was a problem hiding this comment.
Surcharge du répertoire final des assets après compilation (cf. https://symfony.com/doc/current/frontend/asset_mapper.html#serving-assets-in-dev-vs-prod).
vgreb
force-pushed
the
feature/semantic-ui-no-cdn
branch
3 times, most recently
from
fa82a7a to
ab3700f
Compare
vgreb
force-pushed
the
feature/semantic-ui-no-cdn
branch
from
ab3700f to
bd9f68c
Compare
| /** | ||
| * cf. https://symfony.com/doc/current/frontend/asset_mapper.html#serving-assets-in-dev-vs-prod | ||
| */ | ||
| class PostCompilationCopyHandler implements PublicAssetsFilesystemInterface |
There was a problem hiding this comment.
C'est à cause du htdocs ça ou autre chose ?
Si un jour on migre vers un public est-ce que cette classe sera encore nécessaire ?
There was a problem hiding this comment.
Oui c'est en partie à cause de ça mais surtout parce que la commande asset-map:compile se base sur le paramètre %kernel.project_dir% qui dans notre cas est le répertoire app à la racine.
Ainsi, sans surcharge, les fichiers compilés seraient écrits dans app/public.
J'aurai pu surcharger la valeur de %kernel.project_dir% dans le composer.json config.extra.symfony-app-dir mais j'ai préféré éviter pour limiter les effets de bord.
vgreb
force-pushed
the
feature/semantic-ui-no-cdn
branch
from
bd9f68c to
26d9a05
Compare
3 participants
Ajout de semantic-ui pour ne plus dépendre d'un cdn.
close #2175