Fix code review suggestions

app/assets/javascripts/droplab/droplab_ajax.js

@@ -3,6 +3,10 @@
 /* global droplab */
 
 require('../window')(function(w){
+  function droplabAjaxException(message) {
+    this.message = message;
+  }
+
   w.droplabAjax = {
     _loadUrlData: function _loadUrlData(url) {
       return new Promise(function(resolve, reject) {
@@ -56,11 +60,7 @@ require('../window')(function(w){
           }
           hook.list[config.method].call(hook.list, d);
         }).catch(function(e) {
-          if(e.message) {
-            console.error(e.message, e.stack); // eslint-disable-line no-console
-          } else {
-            console.error(e); // eslint-disable-line no-console
-          }
+          throw new droplabAjaxException(e.message || e);
         });
     },
 

app/assets/javascripts/lib/utils/common_utils.js.es6

@@ -169,6 +169,8 @@
     w.gl.utils.getParameterByName = (name) => {
       const url = window.location.href;
       name = name.replace(/[[\]]/g, '\\$&');
+      // Finds the value associated to the name
+      // Example, state=open where state is the name and open is the value
       const regex = new RegExp(`[?&]${name}(=([^&#]*)|&|#|$)`);
       const results = regex.exec(url);
       if (!results) return null;