Disable compiling (minifying) JS files on deploy

... because we don't even use the minified files on production anyway!

This change was motivated because #161 broke
the JS compile step in deploy.sh. Ideally we should fix that and re-enable
minification, but until we re-enable minification there is no point to compiling
JS on deploy.

config/flask_prod.py

@@ -2,7 +2,12 @@
 
 import rmc.shared.secrets as s
 
+# TODO(david): Re-enable minification by changing this directory to 'js_prod'.
+#     Minification was disabled in github.com/UWFlow/rmc/commit/52fae50b
+#     to be able to debug Airbrake issues, but it looks like Airbrake supports
+#     source maps now.
 JS_DIR = 'js'
+
 DEBUG = False
 ENV = 'prod'
 GA_PROPERTY_ID = 'UA-35073503-1'

deploy.sh

@@ -26,8 +26,21 @@ sudo pip install -r requirements.txt
 echo "Compiling compass"
 compass compile server --output-style compressed --force
 
-echo "Compiling js"
-( cd server && node r.js -o build.js )
+# TODO(david): Uncomment the two lines below to re-enable compiling and
+#     minifying JS. This involves doing the following:
+#     1. Fix this compile step: running this will result in an error right now.
+#        This runs the Require.js optimizer (see
+#        http://requirejs.org/docs/optimization.html), which parses out and
+#        tries to run the contents of `require.config({})` in
+#        server/static/js/main.js. However, in github.com/UWFlow/rmc/pull/161 we
+#        moved the raw JSON contents of main.js:require.config() into its own
+#        file, config_settings.js, in order to facilitate a consistent
+#        environment for JS tests.
+#     2. Change JS_DIR in flask_prod.py from 'js' to 'js_prod', where the
+#        minified files end up. This will require producing source maps and
+#        configuring Airbrake to use them.
+#echo "Compiling js"
+#( cd server && node r.js -o build.js )
 
 sudo service rmc_daemon restart
 sudo service celeryd restart