Variety of fixes focusing on login flow and culling errors

Mulligan-Fund · Dec 18, 2017 · 01cb466 · 01cb466
1 parent ae0813f
commit 01cb466
Show file tree

Hide file tree

Showing 7 changed files with 34 additions and 36 deletions.
diff --git a/_config.yml b/_config.yml
@@ -14,7 +14,7 @@
 # You can create any custom variable you would like, and they will be accessible
 # in the templates via {{ site.myvariable }}.
 title: Grant Calculator
-email: your-email@example.com
+email: alb@andrewlb.com
 description: >- # this means to ignore newlines until "baseurl:"
   Write an awesome description for your new site here. You can edit this
   line in _config.yml. It will appear in your document head meta (for

diff --git a/_includes/header.html b/_includes/header.html
@@ -2,7 +2,7 @@
 	<div class="container">
 	  <div class="row">
 	    <div class="logo col col--xs-5 col--sm-5 col--md-5 col--lg-5">
-	      <a href="{{ site.baseurl }}/">
+	      <a href="{{ "/list" | prepend: site.baseurl | prepend: site.url }}">
 	        <img src="{{ "/img/logo.png" | prepend: site.baseurl | prepend: site.url }}" />
 	      </a>
 	    </div>

diff --git a/_layouts/home.html b/_layouts/home.html
@@ -22,7 +22,7 @@ <h2>Find the True Value of your Grant Application</h2>
         <input type="password" id="password">
         <label>Your EIN (optional)</label>
         <input type="text" id="ein">
-        <button class="submit">Register/Login</button>
+        <button class="submit">Register or Login</button>
       </form>
       </div>
     <div class="description">

diff --git a/js/form.js b/js/form.js
@@ -11,6 +11,7 @@ gc.Views = gc.Views || {};
         },
         initialize: function() {
         	var _this = this
+            console.log("Init form",this.el)
             if($(this.el).hasClass('page')) {
                 this.collection.getID()
                 this.getFields()
@@ -19,25 +20,21 @@ gc.Views = gc.Views || {};
 
         submitForm: function(e) {
             var _this = this
-            console.log("Form clicked")
             var l = $(this.el).find('input').length
             var t = {}
 
             $(this.el).find('input').each(function(e,i){
                 t[$(i).attr('id')] = $(i).val()
 
             })
-
-            console.log(t)
-
             this.collection.sendData( 
                 t, function(data,status) {
-                console.log(data)
+                // Redirect
                 if(String(data).match('/')) {
+                    console.log("Redirecting")
                     window.location.replace(data)
                 }
             })
-
         },
 
         submitField: function(e) {

diff --git a/js/grant.js b/js/grant.js
@@ -90,7 +90,8 @@ gc.Collections = gc.Collections || {};
             },
             error : function(request,error)
             {
-                alert("Get Error: "+JSON.stringify(request));
+                // alert("Get Error: "+JSON.stringify(request));
+                callback()
             }
             });
         }

diff --git a/js/main.js b/js/main.js
@@ -19,30 +19,29 @@ var getAPIPath = function getAPIPath() {
 }
 
 function checkAuth(callback) {
-    if(window.location.href !== window.gc.home) {
-        $.ajax({
-            url : window.gc.api + '/auth',
-            type : 'GET',
-            xhrFields: {
-                withCredentials: true
-            },
-            headers: {
-                'Access-Control-Allow-Origin': true
-            },
-            crossDomain: true,  
-            success : function(data,status,xfr) {              
-                console.log("Authenticated");
-                callback()
-            },
-            error : function(request,error)
-            {
-                if(window.location.href !== window.gc.home) window.location.replace(window.gc.home)
-                else console.log("Already logged out")
-            }
-        });
-    } else {
-        callback()
-    }
+    console.log("Checking auth")
+    $.ajax({
+        url : window.gc.api + '/auth',
+        type : 'GET',
+        xhrFields: {
+            withCredentials: true
+        },
+        headers: {
+            'Access-Control-Allow-Origin': true
+        },
+        crossDomain: true,  
+        success : function(data,status,xfr) {              
+            console.log("Authenticated",data);
+            if(window.location.href == window.gc.home) window.location.replace(window.gc.home+'/list')
+            callback()
+        },
+        error : function(request,error)
+        {
+            if(window.location.href !== window.gc.home) window.location.replace(window.gc.home)
+            else console.log("Already logged out")
+            callback()
+        }
+    });
 }
 
 var getUrlParameter = function getUrlParameter(sParam) {

diff --git a/js/user.js b/js/user.js
@@ -45,7 +45,7 @@ gc.Collections = gc.Collections || {};
                 dataType:'json',
                 data: data,
                 success : function(data,status,xfr) {              
-                console.log("Put Success: ",data,status,xfr)
+                    console.log("Put Success: ", data, status, xfr)
                     callback(data,status)
                 },
                 error : function(request,error)
@@ -75,7 +75,8 @@ gc.Collections = gc.Collections || {};
             },
             error : function(request,error)
             {
-                alert("Get Error: "+JSON.stringify(request));
+                // alert("Get Error: "+JSON.stringify(request));
+                callback()
             }
             });
         }