From b6aeeb72705767cf25c0f628deb07fa1403027e5 Mon Sep 17 00:00:00 2001
From: John Obelenus <jobelenus@systeminit.com>
Date: Wed, 18 Dec 2024 18:03:10 -0500
Subject: [PATCH] Fix: load all components at the right time

After first paint, but before the data is required by other things!
---
 app/web/src/store/components.store.ts | 18 ------------------
 app/web/src/store/views.store.ts      |  5 ++---
 2 files changed, 2 insertions(+), 21 deletions(-)

diff --git a/app/web/src/store/components.store.ts b/app/web/src/store/components.store.ts
index d908fc21ea..2f167d90dc 100644
--- a/app/web/src/store/components.store.ts
+++ b/app/web/src/store/components.store.ts
@@ -1427,29 +1427,11 @@ export const useComponentsStore = (forceChangeSetId?: ChangeSetId) => {
             ],
           );
 
-          realtimeStore.subscribe(
-            `${this.$id}-workspace`,
-            `workspace/${workspaceId}`,
-            [
-              {
-                eventType: "ChangeSetApplied",
-                callback: (data) => {
-                  // If the applied change set has rebased into this change set,
-                  // then refetch (i.e. there might be updates!)
-                  if (data.toRebaseChangeSetId === changeSetId) {
-                    this.FETCH_ALL_COMPONENTS();
-                  }
-                },
-              },
-            ],
-          );
-
           const actionUnsub = this.$onAction(handleStoreError);
 
           return () => {
             actionUnsub();
             realtimeStore.unsubscribe(`${this.$id}-changeset`);
-            realtimeStore.unsubscribe(`${this.$id}-workspace`);
           };
         },
       },
diff --git a/app/web/src/store/views.store.ts b/app/web/src/store/views.store.ts
index 9685dcc639..cfec27e701 100644
--- a/app/web/src/store/views.store.ts
+++ b/app/web/src/store/views.store.ts
@@ -650,14 +650,13 @@ export const useViewsStore = (forceChangeSetId?: ChangeSetId) => {
           this.recentViews.push(id);
         },
         async LIST_VIEWS(all = false) {
-          await componentsStore.FETCH_ALL_COMPONENTS();
-
           return new ApiRequest<ViewDescription[]>({
             method: "get",
             url: API_PREFIX,
             onSuccess: async (views) => {
               this.viewList = views;
               this.SORT_LIST_VIEWS();
+              await componentsStore.FETCH_ALL_COMPONENTS();
               for (const { id } of views) {
                 // assuming we're coming from "on load", we have the FETCH that gets the selectedViewId
                 if (all || id !== this.selectedViewId)
@@ -674,6 +673,7 @@ export const useViewsStore = (forceChangeSetId?: ChangeSetId) => {
           });
         },
         async FETCH_VIEW_GEOMETRY(viewId: ViewId) {
+          // requires all components to be in place!
           return new ApiRequest<{
             viewId: ViewId;
             name: string;
@@ -709,7 +709,6 @@ export const useViewsStore = (forceChangeSetId?: ChangeSetId) => {
                 ([componentId, geo]) => {
                   const node = componentsStore.allComponentsById[componentId];
                   if (!node) return;
-                  if (!view) return; // no idea why linting is complaining that i need this
                   let geometry: IRect;
                   if ("width" in node) {
                     geo.width = node.width;