Use safeperson for tutorial

hackclub · Nov 14, 2024 · 12539f3 · 12539f3
1 parent aa73562
commit 12539f3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 9 deletions.
diff --git a/src/app/harbor/tabs/tour.ts b/src/app/harbor/tabs/tour.ts
@@ -2,7 +2,7 @@ import Shepherd, { type Tour } from 'shepherd.js'
 import './shepherd.css'
 import { offset } from '@floating-ui/dom'
 import Cookies from 'js-cookie'
-import { fetchTutorialCompletionStatus } from '../../utils/airtable'
+import { safePerson } from '../../utils/airtable'
 
 const waitForElement = (
   selector: string,
@@ -58,9 +58,10 @@ const t = new Shepherd.Tour({
 
 let hasSetUp = false
 export function tour() {
-  fetchTutorialCompletionStatus().then((res) =>
-    sessionStorage.setItem('tutorial', res.toString()),
-  )
+  safePerson().then(({ hasCompletedTutorial }) => {
+    console.log('Setting tutorial sessionstorage to', hasCompletedTutorial)
+    sessionStorage.setItem('tutorial', hasCompletedTutorial.toString())
+  })
 
   const currentStepId = getCookie('tour-step')
   if (currentStepId) {

diff --git a/src/app/utils/airtable.ts b/src/app/utils/airtable.ts
@@ -151,8 +151,3 @@ export async function safePerson(): Promise<SafePerson> {
     preexistingUser,
   }
 }
-
-export const fetchTutorialCompletionStatus = async (): Promise<boolean> => {
-  const person = await getSelfPerson(slackId)
-  return person.fields.academy_completed === true
-}