fix: update profile interests type to be optional and add guards

open-sauced · Aug 8, 2024 · b59a6c7 · b59a6c7
1 parent df9ae69
commit b59a6c7
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/components/organisms/TopNav/top-nav.tsx b/components/organisms/TopNav/top-nav.tsx
@@ -44,7 +44,7 @@ const TopNav = () => {
 const Nav = ({ className }: { className?: string }) => {
   const { user } = useSupabaseAuth();
   const { data: gitHubUser } = useFetchUser(user?.user_metadata.user_name);
-  const userInterest = gitHubUser?.interests.split(",")[0] || "javascript";
+  const userInterest = gitHubUser?.interests?.split(",")[0] || "javascript";
   const router = useRouter();
 
   const explorePageUrlPattern = /^(\/explore\/topic).*\/(dashboard|reports|contributors|activity).*/g;

diff --git a/components/organisms/UserSettingsPage/user-settings-page.tsx b/components/organisms/UserSettingsPage/user-settings-page.tsx
@@ -143,7 +143,7 @@ const UserSettingsPage = ({ user }: UserSettingsPageProps) => {
         receive_collaboration: insightsUser?.receive_collaboration,
         receive_product_updates: insightsUser?.receive_product_updates,
       });
-      setSelectedInterest(insightsUser?.interests?.split(","));
+      setSelectedInterest(insightsUser?.interests?.split(",") || []);
       setDisplayLocalTime(insightsUser?.display_local_time);
       setTimezone(insightsUser?.timezone || "");
     }

diff --git a/components/shared/AppSidebar/AppSidebar.tsx b/components/shared/AppSidebar/AppSidebar.tsx
@@ -87,7 +87,7 @@ export const AppSideBar = ({ workspaceId, hideSidebar, sidebarCollapsed }: AppSi
   });
 
   const { data: gitHubUser } = useFetchUser(user?.user_metadata.user_name);
-  const userInterest = gitHubUser?.interests.split(",")[0] || "javascript";
+  const userInterest = gitHubUser?.interests?.split(",")[0] || "javascript";
 
   return (
     // TODO: get rid of the z-index. There is grid content like the avatars and paged data text that bleed through the sidebar atm.

diff --git a/next-types.d.ts b/next-types.d.ts
@@ -337,7 +337,7 @@ interface DbUser {
   readonly location: string;
   readonly display_local_time: boolean;
   readonly name: string;
-  readonly interests: string;
+  readonly interests?: string;
   readonly receive_collaboration: boolean;
   readonly display_email: boolean;
   readonly timezone: string;