@homebound/beam 2.244.2 → 2.244.4

package/dist/components/Table/utils/TableState.d.ts

@@ -48,7 +48,7 @@ export declare class TableState {
     setRows(rows: GridDataRow<any>[]): void;
     setColumns(columns: GridColumnWithId<any>[], visibleColumnsStorageKey: string | undefined): void;
     /** Determines which columns to expand immediately vs async */
-    parseAndUpdateExpandedColumns(columnsToExpand: GridColumnWithId<any>[]): void;
+    parseAndUpdateExpandedColumns(columnsToExpand: GridColumnWithId<any>[]): Promise<void>;
     /** Updates the state of which columns are expanded */
     updateExpandedColumns(newColumns: GridColumnWithId<any>[]): void;
     loadExpandedColumns(column: GridColumnWithId<any>): Promise<void>;

package/dist/components/Table/utils/TableState.js

@@ -201,7 +201,7 @@ class TableState {
         }
     }
     /** Determines which columns to expand immediately vs async */
-    parseAndUpdateExpandedColumns(columnsToExpand) {
+    async parseAndUpdateExpandedColumns(columnsToExpand) {
         // Separate out which columns need to be loaded async vs which can be loaded immediately.
         const [localColumnsToExpand, asyncColumnsToExpand] = columnsToExpand.reduce((acc, c) => {
             if ((0, utils_2.isFunction)(c.expandColumns)) {
@@ -212,7 +212,16 @@ class TableState {
         // Handle all async expanding columns using a Promise.all.
         // This will allow the table to render immediately, then cause a rerender with the new columns
         if (asyncColumnsToExpand.length > 0) {
-            Promise.all(asyncColumnsToExpand.map(async (c) => await this.loadExpandedColumns(c))).then(() => this.updateExpandedColumns(asyncColumnsToExpand));
+            // Note: Not using a Promise.all because there seems to be a bug in Apollo with applying TypePolicies when using Promise.all.
+            // TODO: Update comment with Apollo issue link.
+            // Promise.all(asyncColumnsToExpand.map(async (c) => await this.loadExpandedColumns(c))).then(() =>
+            //   this.updateExpandedColumns(asyncColumnsToExpand),
+            // );
+            // Instead, doing each async request in sequence for now.
+            for await (const column of asyncColumnsToExpand) {
+                await this.loadExpandedColumns(column);
+            }
+            this.updateExpandedColumns(asyncColumnsToExpand);
         }
         // For local columns, we skip the Promise in order to have the correct state on the initial load.
         if (localColumnsToExpand.length > 0) {

package/dist/hooks/useComputed.js

@@ -43,6 +43,10 @@ function useComputed(fn, deps) {
         });
         // eslint-disable-next-line react-hooks/exhaustive-deps
     }, deps);
+    // unsubscribe the autorun when we're unmounted
+    (0, react_1.useEffect)(() => {
+        return ref.current.runner;
+    }, []);
     // Occasionally autorun will not have run yet, in which case we have to just
     // accept running the eval fn twice (here to get the value for the 1st render,
     // and again for mobx to watch what observables we touch).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homebound/beam",
-  "version": "2.244.2",
+  "version": "2.244.4",
   "author": "Homebound",
   "license": "MIT",
   "main": "dist/index.js",