fix: let's try a bookmark

skillrecordings · Oct 25, 2024 · ac076e4 · ac076e4
1 parent adf310f
commit ac076e4
Show file tree

Hide file tree

Showing 7 changed files with 28 additions and 17 deletions.
diff --git a/src/components/layouts/collection-page-layout.tsx b/src/components/layouts/collection-page-layout.tsx
@@ -495,9 +495,9 @@ const CollectionPageLayout: React.FunctionComponent<
 
               {/* TODO: fix bookmarks feature EGG-294 */}
               {/* Start of action buttons block */}
-              {/* <div className="flex items-center justify-center mt-4 space-x-2 dark:text-gray-900 md:justify-start"> */}
-              {/* Bookmark button */}
-              {/* {toggle_favorite_url ? (
+              <div className="flex items-center justify-center mt-4 space-x-2 dark:text-gray-900 md:justify-start">
+                {/* Bookmark button */}
+                {toggle_favorite_url ? (
                   <button
                     onClick={() => {
                       if (clickable) {
@@ -559,10 +559,10 @@ const CollectionPageLayout: React.FunctionComponent<
                       </p>
                     </LoginForm>
                   </DialogButton>
-                )} */}
+                )}
 
-              {/* RSS button */}
-              {/* {rss_url ? (
+                {/* RSS button */}
+                {/* {rss_url ? (
                   <Link
                     href={rss_url}
                     onClick={() => {
@@ -592,7 +592,7 @@ const CollectionPageLayout: React.FunctionComponent<
                     </MembershipDialogButton>
                   </a>
                 )} */}
-              {/* </div> */}
+              </div>
               {/* End of action buttons block */}
 
               <div className="flex items-center justify-center w-full mt-5 md:hidden">

diff --git a/src/hooks/use-cio.tsx b/src/hooks/use-cio.tsx
@@ -55,6 +55,7 @@ export const CioProvider: React.FunctionComponent<
         setSubscriber(data)
         cioIdentify(data.id)
       })
+      .catch(() => {})
       .finally(() => setLoadingSubscriber(false))
   }, [])
 

diff --git a/src/lib/playlists.ts b/src/lib/playlists.ts
@@ -155,6 +155,8 @@ export async function loadPlaylist(slug: string, token?: string) {
         published_at
         access_state
         visibility_state
+        toggle_favorite_url
+        favorited
         state
         tags {
           name
@@ -294,6 +296,8 @@ export async function loadPlaylist(slug: string, token?: string) {
     slug: slug,
   }
 
+  console.log('query', token)
+
   const graphQLClient = getGraphQLClient(token)
 
   const {playlist} = await graphQLClient.request(query, variables)

diff --git a/src/pages/api/cio/identify/[id].ts b/src/pages/api/cio/identify/[id].ts
@@ -34,11 +34,13 @@ const cioIdentify = async (req: NextApiRequest, res: NextApiResponse) => {
         Authorization: `Basic ${ENCODED_CUSTOMER_IO_TRACKING_API_CREDENTIALS}`,
       }
 
-      await axios.put(
-        `https://track.customer.io/api/v1/customers/${id}`,
-        {...options, _update: true},
-        {headers},
-      )
+      await axios
+        .put(
+          `https://track.customer.io/api/v1/customers/${id}`,
+          {...options, _update: true},
+          {headers},
+        )
+        .catch(() => {})
 
       const cioCookie = serverCookie.serialize(CIO_IDENTIFIER_KEY, id, {
         secure: process.env.NODE_ENV === 'production',

diff --git a/src/pages/courses/[course]/index.tsx b/src/pages/courses/[course]/index.tsx
@@ -118,7 +118,12 @@ export const getServerSideProps: GetServerSideProps = async ({
 }) => {
   setupHttpTracing({name: getServerSideProps.name, tracer, req, res})
   try {
-    const course = params && (await loadPlaylist(params.course as string))
+    const course =
+      params &&
+      (await loadPlaylist(
+        params.course as string,
+        req.cookies[ACCESS_TOKEN_KEY],
+      ))
 
     const courseSlug = getSlugFromPath(course?.path)
     if (course && courseSlug !== params?.course) {

diff --git a/src/utils/analytics/identify.ts b/src/utils/analytics/identify.ts
@@ -36,7 +36,7 @@ export const identify = async (data: Viewer, properties?: any) => {
       mixpanel.opt_out_tracking()
     }
   } catch (e) {
-    console.error('caught error in identify', e)
+    // console.error('caught error in identify', e)
     return Promise.resolve(false)
   }
 

diff --git a/src/utils/analytics/track.ts b/src/utils/analytics/track.ts
@@ -2,7 +2,6 @@ import {isFunction, isUndefined} from 'lodash'
 import {Viewer} from '@/types'
 import Auth from '../auth'
 import mixpanel from 'mixpanel-browser'
-import posthog from 'posthog-js'
 import {identify} from './identify'
 import PosthogClient from '@/lib/posthog-client'
 const DEBUG_ANALYTICS = true
@@ -77,13 +76,13 @@ export const track = (
         try {
           window._cio.track(event, params)
         } catch (e) {
-          console.error('caught error', e)
+          // console.error('caught error', e)
         }
       }
 
       politelyExit()
     } catch (e) {
-      console.error('caught error', e)
+      // console.error('caught error', e)
       resolve(false)
     }
   })