fix: add console logs when in dev. change webpack target to node

Changing the webpack target to node allows process.env vars to pass through to this built component.
edx · Apr 10, 2019 · a826c19 · a826c19
1 parent 9095aaf
commit a826c19
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/src/LoggingService.js b/src/LoggingService.js
@@ -18,12 +18,18 @@ function fixErrorLength(error) {
 
 class LoggingService {
   static logInfo(message) {
+    if (process.env.NODE_ENV === 'development') {
+      console.log(message); // eslint-disable-line
+    }
     if (typeof window.newrelic !== 'undefined') {
       window.newrelic.addPageAction('INFO', { message });
     }
   }
 
   static logError(error, customAttributes) {
+    if (process.env.NODE_ENV === 'development') {
+      console.error(error, customAttributes); // eslint-disable-line
+    }
     if (typeof window.newrelic !== 'undefined') {
       // Note: customProperties are not sent.  Presumably High-Security Mode is being used.
       window.newrelic.noticeError(fixErrorLength(error), customAttributes);

diff --git a/webpack.config.js b/webpack.config.js
@@ -5,6 +5,10 @@ const UglifyJsPlugin = require('uglifyjs-webpack-plugin');
 
 module.exports = {
   devtool: 'source-map',
+  target: 'node',
+  optimization: {
+    nodeEnv: false,
+  },
   entry: './src/index.js',
   output: {
     path: path.resolve(__dirname, 'dist'),