jquery imported as an extra entry instead of chunk - undefined chunk fix

Oksydan · Nov 3, 2023 · 26d3627 · 26d3627
1 parent d506701
commit 26d3627
Show file tree

Hide file tree

Showing 8 changed files with 9 additions and 12 deletions.
diff --git a/_dev/js/jquery.js b/_dev/js/jquery.js
@@ -0,0 +1 @@
+import 'jquery'; // ONLY FOR CHUNK
diff --git a/_dev/js/theme.js b/_dev/js/theme.js
@@ -1,3 +1 @@
-import 'jquery'; // ONLY FOR CHUNK
-
 import './theme/index';
diff --git a/_dev/js/theme/index.js b/_dev/js/theme/index.js
@@ -2,7 +2,7 @@ import EventEmitter from 'events';
 
 import './windowExpose';
 import './core/index';
-import './vendors/bootstrap/bootstrap-imports';
+import './vendors/index';
 import './components/dynamic-bootstrap-components';
 import bsCustomFileInput from 'bs-custom-file-input';
 import './components/header/index';

diff --git a/_dev/js/theme/vendors/index.js b/_dev/js/theme/vendors/index.js
@@ -0,0 +1 @@
+import './bootstrap/bootstrap-imports';
diff --git a/_dev/webpack/webpack.parts.js b/_dev/webpack/webpack.parts.js
@@ -158,12 +158,6 @@ exports.extractVendorsChunks = () => ({
           filename: 'js/swipervendor.js',
           chunks: 'initial',
         },
-        jquery: {
-          test: /[\\/]node_modules[\\/](jquery)[\\/]/,
-          name: 'jquery',
-          filename: 'js/jquery.js',
-          chunks: 'initial',
-        },
         shared: {
           name: 'shared',
           chunks: 'initial',

diff --git a/_dev/webpack/webpack.production.js b/_dev/webpack/webpack.production.js
@@ -11,6 +11,7 @@ const plugins = (purge, analyze) => ([
   new EsbuildPlugin({
     target: 'es2016',
     format: 'iife',
+    minify: true,
   }),
   analyze ? new BundleAnalyzerPlugin() : false,
   purge ? new PurgeCSSPlugin({

diff --git a/_dev/webpack/webpack.vars.js b/_dev/webpack/webpack.vars.js
@@ -67,6 +67,8 @@ exports.webpackVars = {
       ]
     }
 
+    resultEntries.jquery = path.resolve(themeDev, `./js/jquery.js`);
+
     return resultEntries;
   },
   getOutput: ({ mode, publicPath, siteURL, port, devServer }) => ({

diff --git a/config/assets.yml b/config/assets.yml
@@ -30,9 +30,9 @@ js:
   runtime: # This file is only used to handle HMR for multiple entry points
     fileName: runtime.js
     priority: 1
-  jquery:
-    fileName: jquery.js
-    priority: 1
+#  jquery: # Uncomment this if you want to use jQuery
+#    fileName: jquery.js
+#    priority: 1
   shared: # Shared code between all pages autogenerated by Webpack
     fileName: shared.js
     priority: 1
Original file line number	Diff line number	Diff line change
		@@ -1,3 +1 @@
		import 'jquery'; // ONLY FOR CHUNK

		import './theme/index';
-Original file line number
+Diff line change
@@ Expand Up / @@ -67,6 +67,8 @@ exports.webpackVars = { @@
           ]
         }
+        resultEntries.jquery = path.resolve(themeDev, `./js/jquery.js`);
         return resultEntries;
       },
       getOutput: ({ mode, publicPath, siteURL, port, devServer }) => ({
@@ Expand Down @@