From f43d03393babc939756c9f5581343913b02c7298 Mon Sep 17 00:00:00 2001 From: n-peugnet Date: Thu, 7 Nov 2019 14:02:13 +0100 Subject: build: ENV prod enhancements - fix: prod mode not working with target-specific ENV variable - use another sourcemap mode in dev mode - generate souremap files in prod mode --- webpack.config.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'webpack.config.js') diff --git a/webpack.config.js b/webpack.config.js index b4a445a..1293e7e 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -5,7 +5,7 @@ module.exports = (env) => { return { // Environment dependent mode: env == 'dev' ? 'development' : 'production', - devtool: env == 'dev' ? 'inline-source-map' : 'none', + devtool: env == 'dev' ? 'cheap-eval-source-map' : 'hidden-source-map', stats: env == 'dev' ? {} : { warnings: false }, // Constant -- cgit v1.2.3 From 6eddd69843d78bf442324dd10c501b79bbaf89d1 Mon Sep 17 00:00:00 2001 From: n-peugnet Date: Thu, 7 Nov 2019 17:58:08 +0100 Subject: feat: include sentry in frontend --- webpack.config.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'webpack.config.js') diff --git a/webpack.config.js b/webpack.config.js index 1293e7e..02d9318 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -5,13 +5,14 @@ module.exports = (env) => { return { // Environment dependent mode: env == 'dev' ? 'development' : 'production', - devtool: env == 'dev' ? 'cheap-eval-source-map' : 'hidden-source-map', + devtool: env == 'dev' ? 'cheap-eval-source-map' : 'source-map', stats: env == 'dev' ? {} : { warnings: false }, // Constant entry: { edit: './src/edit.js', home: './src/home.js', + sentry: './src/sentry.js', }, output: { filename: 'assets/js/[name].bundle.js', -- cgit v1.2.3 From e4bc65433283e0725a2e7bf07ae0c84f7905af17 Mon Sep 17 00:00:00 2001 From: n-peugnet Date: Mon, 11 Nov 2019 18:50:56 +0100 Subject: feat: make Sentry optional - move sentry to PHP dev requirements. - try...catch Sentry integration in PHP. - update Makefile: - add comments as documentation. - remove recursive call from PREV_ENV_FILE target. - make dist and sentryrelease ENV variable. - add touch target for more efficient rebuilds. - add `dist` build ENV functionnality not to include dev-requirements in the zip release. --- webpack.config.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'webpack.config.js') diff --git a/webpack.config.js b/webpack.config.js index 02d9318..8cd7764 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -5,7 +5,11 @@ module.exports = (env) => { return { // Environment dependent mode: env == 'dev' ? 'development' : 'production', - devtool: env == 'dev' ? 'cheap-eval-source-map' : 'source-map', + devtool: env == 'dev' ? + 'cheap-eval-source-map' : + env == 'dist' ? + 'hidden-source-map' : + 'source-map', stats: env == 'dev' ? {} : { warnings: false }, // Constant -- cgit v1.2.3 From c1843c0a2ef49de3d2030398f5c14549cc1791ab Mon Sep 17 00:00:00 2001 From: n-peugnet Date: Fri, 15 Nov 2019 15:57:46 +0100 Subject: feat: load Sentry browser as an external script - better way to check if we need to report errors in PHP - fix make file $(js_bundles) dependencies (add webpack conf) - update sentry-browser sdk and move it to dev requirements --- webpack.config.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'webpack.config.js') diff --git a/webpack.config.js b/webpack.config.js index 8cd7764..9c6607d 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -19,8 +19,8 @@ module.exports = (env) => { sentry: './src/sentry.js', }, output: { - filename: 'assets/js/[name].bundle.js', - path: path.resolve(__dirname), + filename: '[name].bundle.js', + path: path.resolve(__dirname, 'assets', 'js'), libraryTarget: 'window' }, module: { @@ -41,5 +41,8 @@ module.exports = (env) => { singleQuote: true, }) ], + externals: { + '@sentry/browser': 'Sentry', + }, } }; -- cgit v1.2.3