fix: profile page crash (#8596)

* fix: profile page crash * fix: update
codesandbox · Sep 9, 2024 · 3e9d4e5 · 3e9d4e5
1 parent 94e7994
commit 3e9d4e5
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 7 deletions.
diff --git a/packages/app/src/app/overmind/namespaces/profile/actions.ts b/packages/app/src/app/overmind/namespaces/profile/actions.ts
@@ -20,6 +20,17 @@ export const profileMounted = withLoadApp(
 
     state.profile.profiles[profile.id] = profile;
     state.profile.currentProfileId = profile.id;
+
+    if (profile.showcasedSandboxShortid) {
+      try {
+        state.profile.showcasedSandbox = await effects.api.getSandbox(
+          profile.showcasedSandboxShortid
+        );
+      } catch (e) {
+        // Ignore it
+      }
+    }
+
     state.profile.isLoadingProfile = false;
   }
 );
@@ -136,8 +147,11 @@ export const newSandboxShowcaseSelected = async (
     return;
   }
 
-  await effects.api.updateShowcasedSandbox(state.user.username, id);
-
+  const sandbox = await effects.api.updateShowcasedSandbox(
+    state.user.username,
+    id
+  );
+  state.profile.showcasedSandbox = sandbox as Sandbox;
   state.profile.isLoadingProfile = false;
 };
 

diff --git a/packages/app/src/app/overmind/namespaces/profile/state.ts b/packages/app/src/app/overmind/namespaces/profile/state.ts
@@ -72,6 +72,7 @@ export const state: State = {
   currentSortDirection: 'desc',
   contextMenu: { sandboxId: null, sandboxType: null, position: null },
   collections: [],
+  showcasedSandbox: null,
   isProfileCurrentUser: derived(
     (currentState: State, rootState: Context['state']) =>
       Boolean(
@@ -83,11 +84,7 @@ export const state: State = {
       ? currentState.profiles[currentState.currentProfileId]
       : null
   ),
-  showcasedSandbox: derived((currentState: State) =>
-    currentState.current && currentState.current.showcasedSandboxShortid
-      ? currentState.sandboxes.all[currentState.current.showcasedSandboxShortid]
-      : null
-  ),
+
   currentLikedSandboxes: derived((currentState: State) =>
     currentState.current
       ? currentState.likedSandboxes[currentState.current.username]