NGSTACK-870 implement vite for asset bundleing instead of webpack#196
NGSTACK-870 implement vite for asset bundleing instead of webpack#196markosandalj wants to merge 1 commit intomasterfrom
Conversation
|
|
||
| // Accessibility | ||
| @import 'accessibility'; | ||
| @import 'accessibility'; No newline at end of file |
| "netgen/layouts-ibexa-relation-list-query": "^1.4", | ||
| "netgen/layouts-ibexa-tags-query": "^1.4" | ||
| "netgen/layouts-ibexa-tags-query": "^1.4", | ||
| "pentatrion/vite-bundle": "^6.4" |
There was a problem hiding this comment.
Can you please move this just below `"tanoconsulting/ibexa-migration-bundle", to keep extra dependencies not related to Netgen together?
| @@ -1,84 +1,90 @@ | |||
| { | |||
| "browserslist": { | |||
There was a problem hiding this comment.
Please revert back to 4 spaces, for more relevant diffs and easier comparison to Ibexa version of package.json
| @@ -1,4 +1,61 @@ | |||
| { | |||
| "babdev/pagerfanta-bundle": { | |||
There was a problem hiding this comment.
Do not commit this change to symfony.lock. Full symfony.lock file is only commited in projects. Here, this file exists only with minimal data to prevent Symfony Flex from destroying the repo on first install, but still allowing fresh recipes to be installed when instantiating a project.
| @@ -0,0 +1,32 @@ | |||
| import { defineConfig, loadEnv } from 'vite'; | |||
There was a problem hiding this comment.
I suppose we can now remove webpack.config.js and rename webpack.config.ibexa.js to webpack.config.js as is in the default Ibexa install. Install docs also need to be updated to remove/adapt the section for renamed webpack config file.
There was a problem hiding this comment.
Together with this, Netgen specific webpack config can be reverted back to Ibexa config in config/packages and removed in config/app/packages.
2 participants
No description provided.