Handling missing name

leighhalliday · Sep 10, 2019 · c5233fd · c5233fd · vercel · Sep 10, 2019
1 parent 4f52dbf
commit c5233fd
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/pages/api/auth/connect.ts b/pages/api/auth/connect.ts
@@ -39,9 +39,11 @@ async function getUser(accessToken) {
 }
 
 async function insertUser(data: GithubData): Promise<User | null> {
+  const username: string = data.login.toLowerCase();
+
   await db("users").insert({
-    name: data.name,
-    username: data.login.toLowerCase(),
+    name: data.name || username,
+    username: username,
     github_id: data.id,
     website_url: data.blog,
     github_url: data.html_url,
@@ -52,10 +54,12 @@ async function insertUser(data: GithubData): Promise<User | null> {
 }
 
 async function updateUser(data: GithubData): Promise<User> {
+  const username: string = data.login.toLowerCase();
+
   await db("users")
     .where({ github_id: data.id })
     .update({
-      name: data.name,
+      name: data.name || username,
       website_url: data.blog,
       github_url: data.html_url,
       updated_at: new Date().toISOString()