@linzjs/step-ag-grid 23.0.1 → 24.0.0

package/src/contexts/GridContext.tsx

@@ -1,8 +1,8 @@
-import { ColDef, GridApi, IRowNode } from "ag-grid-community";
-import { CsvExportParams } from "ag-grid-community";
-import { createContext, useContext } from "react";
+import { ColDef, GridApi, IRowNode } from 'ag-grid-community';
+import { CsvExportParams } from 'ag-grid-community';
+import { createContext, useContext } from 'react';
 
-import { ColDefT, GridBaseRow } from "../components";
+import { ColDefT, GridBaseRow } from '../components';
 
 export type GridFilterExternal<TData extends GridBaseRow> = (data: TData, rowNode: IRowNode) => boolean;
 
@@ -17,7 +17,7 @@ export type AutoSizeColumnsResult = { width: number } | null;
 
 export interface GridContextType<TData extends GridBaseRow> {
   gridReady: boolean;
-  gridRenderState: () => null | "empty" | "rows-visible";
+  gridRenderState: () => null | 'empty' | 'rows-visible';
   getColDef: (colId?: string) => ColDef | undefined;
   getColumns: (
     filter?: keyof ColDef | ((r: ColDef) => boolean | undefined | null | number | string),
@@ -72,138 +72,144 @@ export const GridContext = createContext<GridContextType<any>>({
   gridReady: false,
   gridRenderState: () => null,
   getColDef: () => {
-    console.error("no context provider for getColDef");
+    console.error('no context provider for getColDef');
     return undefined;
   },
   getColumns: () => {
-    console.error("no context provider for getColumns");
+    console.error('no context provider for getColumns');
     return [];
   },
   getColumnIds: () => {
-    console.error("no context provider for getColumnIds");
+    console.error('no context provider for getColumnIds');
     return [];
   },
   invisibleColumnIds: undefined,
   setInvisibleColumnIds: () => {
-    console.error("no context provider for setInvisibleColumnIds");
+    console.error('no context provider for setInvisibleColumnIds');
   },
   prePopupOps: () => {
-    console.error("no context provider for prePopupOps");
+    console.error('no context provider for prePopupOps');
   },
   postPopupOps: () => {
-    console.error("no context provider for prePopupOps");
+    console.error('no context provider for prePopupOps');
   },
   externallySelectedItemsAreInSync: false,
   setApis: () => {
-    console.error("no context provider for setApis");
+    console.error('no context provider for setApis');
   },
   setQuickFilter: () => {
-    console.error("no context provider for setQuickFilter");
+    console.error('no context provider for setQuickFilter');
   },
   selectRowsById: () => {
-    console.error("no context provider for selectRows");
+    console.error('no context provider for selectRows');
   },
-  getSelectedRows: <T extends unknown>(): T[] => {
-    console.error("no context provider for getSelectedRows");
+  getSelectedRows: <T,>(): T[] => {
+    console.error('no context provider for getSelectedRows');
     return [];
   },
-  getFilteredSelectedRows: <T extends unknown>(): T[] => {
-    console.error("no context provider for getFilteredSelectedRows");
+  getFilteredSelectedRows: <T,>(): T[] => {
+    console.error('no context provider for getFilteredSelectedRows');
     return [];
   },
   getSelectedRowIds: () => {
-    console.error("no context provider for getSelectedRowIds");
+    console.error('no context provider for getSelectedRowIds');
     return [];
   },
   getFilteredSelectedRowIds: () => {
-    console.error("no context provider for getFilteredSelectedRowIds");
+    console.error('no context provider for getFilteredSelectedRowIds');
     return [];
   },
+  // eslint-disable-next-line @typescript-eslint/require-await
   selectRowsDiff: async () => {
-    console.error("no context provider for selectRowsDiff");
+    console.error('no context provider for selectRowsDiff');
   },
   selectRowsByIdWithFlash: () => {
-    console.error("no context provider for selectRowsWithFlash");
+    console.error('no context provider for selectRowsWithFlash');
   },
+  // eslint-disable-next-line @typescript-eslint/require-await
   selectRowsWithFlashDiff: async () => {
-    console.error("no context provider for selectRowsWithFlashDiff");
+    console.error('no context provider for selectRowsWithFlashDiff');
   },
   flashRows: () => {
-    console.error("no context provider for flashRows");
+    console.error('no context provider for flashRows');
   },
+  // eslint-disable-next-line @typescript-eslint/require-await
   flashRowsDiff: async () => {
-    console.error("no context provider for flashRows");
+    console.error('no context provider for flashRows');
   },
+  // eslint-disable-next-line @typescript-eslint/require-await,@typescript-eslint/no-misused-promises
   focusByRowById: async () => {
-    console.error("no context provider for focusByRowById");
+    console.error('no context provider for focusByRowById');
   },
   ensureRowVisible: () => {
-    console.error("no context provider for ensureRowVisible");
+    console.error('no context provider for ensureRowVisible');
     return true;
   },
   ensureSelectedRowIsVisible: () => {
-    console.error("no context provider for ensureSelectedRowIsVisible");
+    console.error('no context provider for ensureSelectedRowIsVisible');
   },
   getFirstRowId: () => {
-    console.error("no context provider for getFirstRowId");
+    console.error('no context provider for getFirstRowId');
     return -1;
   },
   autoSizeColumns: () => {
-    console.error("no context provider for autoSizeColumns");
+    console.error('no context provider for autoSizeColumns');
     return null;
   },
   sizeColumnsToFit: () => {
-    console.error("no context provider for autoSizeAllColumns");
+    console.error('no context provider for autoSizeAllColumns');
     return null;
   },
   editingCells: () => {
-    console.error("no context provider for editingCells");
+    console.error('no context provider for editingCells');
     return false;
   },
   cancelEdit: () => {
-    console.error("no context provider for cancelEdit");
+    console.error('no context provider for cancelEdit');
   },
   startCellEditing: () => {
-    console.error("no context provider for startCellEditing");
+    console.error('no context provider for startCellEditing');
   },
   stopEditing: () => {
-    console.error("no context provider for stopEditing");
+    console.error('no context provider for stopEditing');
   },
+  // eslint-disable-next-line @typescript-eslint/require-await
   updatingCells: async () => {
-    console.error("no context provider for modifyUpdating");
+    console.error('no context provider for modifyUpdating');
     return false;
   },
   redrawRows: () => {
-    console.error("no context provider for redrawRows");
+    console.error('no context provider for redrawRows');
   },
   setExternallySelectedItemsAreInSync: () => {
-    console.error("no context provider for setExternallySelectedItemsAreInSync");
+    console.error('no context provider for setExternallySelectedItemsAreInSync');
   },
+  // eslint-disable-next-line @typescript-eslint/require-await
   waitForExternallySelectedItemsToBeInSync: async () => {
-    console.error("no context provider for waitForExternallySelectedItemsToBeInSync");
+    console.error('no context provider for waitForExternallySelectedItemsToBeInSync');
   },
   addExternalFilter: () => {
-    console.error("no context provider for addExternalFilter");
+    console.error('no context provider for addExternalFilter');
   },
   removeExternalFilter: () => {
-    console.error("no context provider for removeExternalFilter");
+    console.error('no context provider for removeExternalFilter');
   },
   isExternalFilterPresent: () => {
-    console.error("no context provider for isExternalFilterPresent");
+    console.error('no context provider for isExternalFilterPresent');
     return false;
   },
   doesExternalFilterPass: () => {
-    console.error("no context provider for doesExternalFilterPass");
+    console.error('no context provider for doesExternalFilterPass');
     return true;
   },
   downloadCsv: () => {
-    console.error("no context provider for downloadCsv");
+    console.error('no context provider for downloadCsv');
   },
   setOnCellEditingComplete: () => {
-    console.error("no context provider for setOnCellEditingComplete");
+    console.error('no context provider for setOnCellEditingComplete');
   },
   showNoRowsOverlay: () => {
-    console.error("no context provider for showLoadingOverlay");
+    console.error('no context provider for showLoadingOverlay');
   },
 });
 

package/src/contexts/GridContextProvider.test.tsx

@@ -1,16 +1,16 @@
-import { ValueFormatterParams } from "ag-grid-community";
+import { ValueFormatterParams } from 'ag-grid-community';
 
-import { downloadCsvUseValueFormattersProcessCellCallback as Dpcc } from "./GridContextProvider";
+import { downloadCsvUseValueFormattersProcessCellCallback as Dpcc } from './GridContextProvider';
 
-describe("downloadCsvUseValueFormattersProcessCellCallback", () => {
-  test("each type of value for conversion to string", async () => {
-    expect(Dpcc({ value: undefined } as any)).toBe("");
-    expect(Dpcc({ value: "-" } as any)).toBe("");
-    expect(Dpcc({ value: "–" } as any)).toBe("");
-    expect(Dpcc({ value: null } as any)).toBe("");
-    expect(Dpcc({ value: false } as any)).toBe("false");
-    expect(Dpcc({ value: "x" } as any)).toBe("x");
-    expect(Dpcc({ value: 123 } as any)).toBe("123");
+describe('downloadCsvUseValueFormattersProcessCellCallback', () => {
+  test('each type of value for conversion to string', () => {
+    expect(Dpcc({ value: undefined } as any)).toBe('');
+    expect(Dpcc({ value: '-' } as any)).toBe('');
+    expect(Dpcc({ value: '–' } as any)).toBe('');
+    expect(Dpcc({ value: null } as any)).toBe('');
+    expect(Dpcc({ value: false } as any)).toBe('false');
+    expect(Dpcc({ value: 'x' } as any)).toBe('x');
+    expect(Dpcc({ value: 123 } as any)).toBe('123');
     expect(Dpcc({ value: { a: 2 } } as any)).toBe('{"a":2}');
     expect(Dpcc({ value: { a: 2 }, column: { getColDef: () => undefined } } as any)).toBe('{"a":2}');
     expect(Dpcc({ value: { a: 2 }, column: { getColDef: () => ({ valueFormatter: undefined }) } } as any)).toBe(
@@ -19,7 +19,7 @@ describe("downloadCsvUseValueFormattersProcessCellCallback", () => {
     expect(
       Dpcc({
         value: { a: 2 },
-        column: { getColDef: () => ({ valueFormatter: "registeredValueFormatter" }) },
+        column: { getColDef: () => ({ valueFormatter: 'registeredValueFormatter' }) },
       } as any),
     ).toBe('{"a":2}');
 
@@ -29,11 +29,11 @@ describe("downloadCsvUseValueFormattersProcessCellCallback", () => {
         column: {
           getColDef: () => ({
             valueFormatter: (params: ValueFormatterParams) => {
-              return "xxx" + params.value.a;
+              return 'xxx' + params.value.a;
             },
           }),
         },
       } as any),
-    ).toBe("xxx2");
+    ).toBe('xxx2');
   });
 });

package/src/contexts/GridContextProvider.tsx

@@ -1,15 +1,15 @@
-import { CellPosition, ColDef, GridApi, IRowNode, RowNode } from "ag-grid-community";
-import { CsvExportParams, ProcessCellForExportParams } from "ag-grid-community";
-import debounce from "debounce-promise";
-import { compact, defer, delay, difference, filter, isEmpty, last, pull, remove, sortBy, sumBy } from "lodash-es";
-import { PropsWithChildren, ReactElement, useCallback, useContext, useEffect, useMemo, useRef, useState } from "react";
-
-import { ColDefT, GridBaseRow } from "../components";
-import { GridCellFillerColId, isGridCellFiller } from "../components/GridCellFiller";
-import { getColId, isFlexColumn } from "../components/gridUtil";
-import { fnOrVar, isNotEmpty, sanitiseFileName, wait } from "../utils/util";
-import { AutoSizeColumnsProps, AutoSizeColumnsResult, GridContext, GridFilterExternal } from "./GridContext";
-import { GridUpdatingContext } from "./GridUpdatingContext";
+import { CellPosition, ColDef, GridApi, IRowNode, RowNode } from 'ag-grid-community';
+import { CsvExportParams, ProcessCellForExportParams } from 'ag-grid-community';
+import debounce from 'debounce-promise';
+import { compact, defer, delay, difference, filter, isEmpty, last, pull, remove, sortBy, sumBy } from 'lodash-es';
+import { PropsWithChildren, ReactElement, useCallback, useContext, useEffect, useMemo, useRef, useState } from 'react';
+
+import { ColDefT, GridBaseRow } from '../components';
+import { GridCellFillerColId, isGridCellFiller } from '../components/GridCellFiller';
+import { getColId, isFlexColumn } from '../components/gridUtil';
+import { fnOrVar, isNotEmpty, sanitiseFileName, wait } from '../utils/util';
+import { AutoSizeColumnsProps, AutoSizeColumnsResult, GridContext, GridFilterExternal } from './GridContext';
+import { GridUpdatingContext } from './GridUpdatingContext';
 
 /**
  * Context for AgGrid operations.
@@ -20,7 +20,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
   const { modifyUpdating, checkUpdating } = useContext(GridUpdatingContext);
   const [gridApi, setGridApi] = useState<GridApi>();
   const [gridReady, setGridReady] = useState(false);
-  const [quickFilter, _setQuickFilter] = useState("");
+  const [quickFilter, _setQuickFilter] = useState('');
   const [invisibleColumnIds, _setInvisibleColumnIds] = useState<string[]>();
   const testId = useRef<string | undefined>();
   const idsBeforeUpdate = useRef<number[]>([]);
@@ -47,7 +47,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
    * Set quick filter directly on grid, based on previously save quickFilter state.
    */
   useEffect(() => {
-    gridApi?.setGridOption("quickFilterText", quickFilter);
+    gridApi?.setGridOption('quickFilterText', quickFilter);
   }, [gridApi, quickFilter]);
 
   /**
@@ -57,7 +57,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
    * @param noApiFn Execute if api is not ready.
    */
   const gridApiOp = useCallback(
-    <T extends unknown, R extends unknown>(hasApiFn: (gridApi: GridApi) => T, noApiFn?: () => R): T | R => {
+    <T, R>(hasApiFn: (gridApi: GridApi) => T, noApiFn?: () => R): T | R => {
       if (!noApiFn) {
         noApiFn = (() => {}) as () => R;
       }
@@ -87,9 +87,9 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
     let id = 0;
     try {
       gridApi?.forEachNodeAfterFilterAndSort((rowNode) => {
-        id = parseInt(rowNode.id ?? "0");
+        id = parseInt(rowNode.id ?? '0');
         // this is the only way to get out of the loop
-        throw "expected exception - exit_loop";
+        throw 'expected exception - exit_loop';
       });
     } catch (ex) {
       // ignore
@@ -104,7 +104,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
     (gridApi: GridApi | undefined, dataTestId?: string) => {
       testId.current = dataTestId;
       setGridApi(gridApi);
-      gridApi?.setGridOption("quickFilterText", quickFilter);
+      gridApi?.setGridOption('quickFilterText', quickFilter);
       setGridReady(!!gridApi);
     },
     [quickFilter],
@@ -113,10 +113,10 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
   /**
    * Used to check if it's OK to autosize.
    */
-  const gridRenderState = useCallback((): null | "empty" | "rows-visible" => {
+  const gridRenderState = useCallback((): null | 'empty' | 'rows-visible' => {
     if (!gridApi) return null;
-    if (!gridApi.getDisplayedRowCount()) return "empty";
-    if (!isEmpty(gridApi.getRenderedNodes())) return "rows-visible";
+    if (!gridApi.getDisplayedRowCount()) return 'empty';
+    if (!isEmpty(gridApi.getRenderedNodes())) return 'rows-visible';
     // If there are rows to render, but there are no rendered nodes then we should wait
     return null;
   }, [gridApi]);
@@ -182,7 +182,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
   const _getNewNodes = useCallback((): IRowNode[] => {
     return gridApiOp(
       (gridApi) =>
-        compact(difference(_getAllRowIds(), idsBeforeUpdate.current).map((rowId) => gridApi.getRowNode("" + rowId))),
+        compact(difference(_getAllRowIds(), idsBeforeUpdate.current).map((rowId) => gridApi.getRowNode('' + rowId))),
       () => [],
     );
   }, [_getAllRowIds, gridApiOp]);
@@ -196,7 +196,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
   const _rowIdsToNodes = useCallback(
     (rowIds: number[]): IRowNode[] => {
       return gridApiOp(
-        (gridApi) => compact(rowIds.map((rowId) => gridApi.getRowNode("" + rowId))),
+        (gridApi) => compact(rowIds.map((rowId) => gridApi.getRowNode('' + rowId))),
         () => [] as RowNode[],
       );
     },
@@ -362,14 +362,14 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
     [_selectRowsWithOptionalFlash],
   );
 
-  const getSelectedRows = useCallback(<T extends unknown>(): T[] => {
+  const getSelectedRows = useCallback(<T,>(): T[] => {
     return gridApiOp(
       (gridApi) => gridApi.getSelectedRows(),
       () => [],
     );
   }, [gridApiOp]);
 
-  const getFilteredSelectedRows = useCallback(<T extends unknown>(): T[] => {
+  const getFilteredSelectedRows = useCallback(<T,>(): T[] => {
     return gridApiOp((gridApi) => {
       const rowData: T[] = [];
       gridApi.forEachNodeAfterFilter((node) => {
@@ -431,7 +431,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
       return {
         width: sumBy(
           gridApi.getColumnState().filter((col) => !col.hide),
-          "width",
+          'width',
         ),
       };
     },
@@ -454,7 +454,8 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
   }, [gridApi]);
 
   const startCellEditing = useCallback(
-    async ({ rowId, colId }: { rowId: number; colId: string }) => {
+    // eslint-disable-next-line @typescript-eslint/require-await
+    ({ rowId, colId }: { rowId: number; colId: string }) => {
       if (!gridApi) return;
 
       const colDef = gridApi.getColumnDef(colId);
@@ -471,7 +472,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
       }
 
       // Cell already being edited, so don't re-edit until finished
-      if (checkUpdating([colDef.field ?? ""], rowId)) {
+      if (checkUpdating([colDef.field ?? ''], rowId)) {
         return;
       }
 
@@ -565,13 +566,13 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
         let ok = false;
 
         await modifyUpdating(
-          props.field ?? "",
+          props.field ?? '',
           selectedRows.map((data) => data.id),
           async () => {
             // Need to refresh to get spinners to work on all rows
             gridApi.refreshCells({ rowNodes: props.selectedRows as RowNode[], force: true });
             ok = await fnUpdate(selectedRows).catch((ex) => {
-              console.error("Exception during modifyUpdating", ex);
+              console.error('Exception during modifyUpdating', ex);
               return false;
             });
           },
@@ -611,14 +612,14 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
     [gridApiOp, modifyUpdating, selectNextEditableCell],
   );
 
-  const redrawRows = useMemo(
+  const redrawRows: any = useMemo(
     () =>
       debounce((rowNodes?: IRowNode[]) => {
         try {
           gridApi && gridApi.redrawRows(rowNodes ? { rowNodes } : undefined);
         } catch (ex) {
           // Hide errors in jest, but log them in browser
-          if (typeof jest === "undefined") console.error(ex);
+          if (typeof jest === 'undefined') console.error(ex);
         }
       }, 50),
     [gridApi],
@@ -646,7 +647,7 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
           gridApi?.forEachNodeAfterFilter((rowNode) => {
             arr.push(rowNode.id);
           });
-          return arr.join("|");
+          return arr.join('|');
         };
 
         if (gridApi) {
@@ -664,12 +665,12 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
 
   const addExternalFilter = (filter: GridFilterExternal<TData>) => {
     externalFilters.current.push(filter);
-    onFilterChanged().then();
+    void onFilterChanged();
   };
 
   const removeExternalFilter = (filter: GridFilterExternal<TData>) => {
     remove(externalFilters.current, (v) => v === filter);
-    onFilterChanged().then();
+    void onFilterChanged();
   };
 
   const isExternalFilterPresent = (): boolean => !isEmpty(externalFilters.current);
@@ -798,10 +799,10 @@ export const GridContextProvider = <TData extends GridBaseRow>(props: PropsWithC
 export const downloadCsvUseValueFormattersProcessCellCallback = (params: ProcessCellForExportParams): string => {
   const encodeToString = (value: any): string => {
     // Convert nullish values to blank
-    if (value === "-" || value === "–" || value == null) {
-      return "";
+    if (value === '-' || value === '–' || value == null) {
+      return '';
     }
-    if (typeof value === "string") {
+    if (typeof value === 'string') {
       return value;
     }
     return JSON.stringify(value);
@@ -816,14 +817,14 @@ export const downloadCsvUseValueFormattersProcessCellCallback = (params: Process
   const valueFormatter = colDef.valueFormatter;
   // If no valueFormatter then value _must_ be a string
   if (valueFormatter == null) {
-    if (params.value != null && typeof params.value !== "string") {
+    if (params.value != null && typeof params.value !== 'string') {
       console.error(`downloadCsv: valueFormatter missing and getValue is not a string, colId: ${colDef.colId}`);
     }
     return encodeToString(params.value);
   }
 
   // We don't have access to registered functions, so we can't call them
-  if (typeof valueFormatter !== "function") {
+  if (typeof valueFormatter !== 'function') {
     console.error(
       `downloadCsv: String type (registered) value formatters are unsupported in downloadCsv, colId: ${colDef.colId}`,
     );
@@ -832,7 +833,7 @@ export const downloadCsvUseValueFormattersProcessCellCallback = (params: Process
 
   const result = valueFormatter({ ...params, data: params.node?.data, colDef, node: null });
   // type may not be string due to casting, leave the type check in
-  if (params.value != null && typeof result !== "string") {
+  if (params.value != null && typeof result !== 'string') {
     console.error(`downloadCsv: valueFormatter is returning non string values, colDef:", colId: ${colDef.colId}`);
   }
   // We add an extra encodeToString here just in case valueFormatter is returning non string values

package/src/contexts/GridPopoverContext.tsx

@@ -1,6 +1,6 @@
-import { RefObject, createContext, useContext } from "react";
+import { createContext, RefObject, useContext } from 'react';
 
-import { GridBaseRow } from "../components/Grid";
+import { GridBaseRow } from '../components/Grid';
 
 export interface GridPopoverContextType<TData extends GridBaseRow> {
   anchorRef: RefObject<Element>;
@@ -11,7 +11,10 @@ export interface GridPopoverContextType<TData extends GridBaseRow> {
   value: any;
   data: TData;
   selectedRows: TData[];
-  updateValue: (saveFn: (selectedRows: any[]) => Promise<boolean>, tabDirection: 1 | 0 | -1) => Promise<boolean>;
+  updateValue: (
+    saveFn: (selectedRows: any[]) => Promise<boolean>,
+    tabDirection: 1 | 0 | -1,
+  ) => Promise<boolean> | boolean;
   formatValue: (value: any) => any;
 }
 
@@ -19,13 +22,13 @@ export const GridPopoverContext = createContext<GridPopoverContextType<any>>({
   anchorRef: { current: null },
   saving: false,
   setSaving: () => {},
-  colId: "",
-  field: "",
+  colId: '',
+  field: '',
   value: null,
   data: {} as GridBaseRow,
   selectedRows: [],
-  updateValue: async () => false,
-  formatValue: () => "! No gridPopoverContextProvider !",
+  updateValue: () => false,
+  formatValue: () => '! No gridPopoverContextProvider !',
 });
 
 export const useGridPopoverContext = <TData extends GridBaseRow>() =>

package/src/contexts/GridPopoverContextProvider.tsx

@@ -1,17 +1,19 @@
-import { ICellEditorParams } from "ag-grid-community";
-import { sortBy } from "lodash-es";
-import { PropsWithChildren, RefObject, useCallback, useContext, useMemo, useRef, useState } from "react";
+import { ICellEditorParams } from 'ag-grid-community';
+import { sortBy } from 'lodash-es';
+import { PropsWithChildren, RefObject, useCallback, useContext, useMemo, useRef, useState } from 'react';
 
-import { GridBaseRow } from "../components/Grid";
-import { GridContext } from "./GridContext";
-import { GridPopoverContext } from "./GridPopoverContext";
+import { GridBaseRow } from '../components/Grid';
+import { GridContext } from './GridContext';
+import { GridPopoverContext } from './GridPopoverContext';
 
 interface GridPopoverContextProps {
-  props: ICellEditorParams;
+  props: ICellEditorParams<any, any, any>;
 }
 
-export const GridPopoverContextProvider = ({ props, children }: PropsWithChildren<GridPopoverContextProps>) => {
+export const GridPopoverContextProvider = (props2: PropsWithChildren<GridPopoverContextProps>) => {
   const { getFilteredSelectedRows, updatingCells } = useContext(GridContext);
+  const children = props2.children;
+  const props = props2.props;
   const anchorRef = useRef<Element>(props.eGridCell);
 
   const hasSaved = useRef(false);
@@ -27,8 +29,8 @@ export const GridPopoverContextProvider = ({ props, children }: PropsWithChildre
     [getFilteredSelectedRows, multiEdit, props.data],
   );
 
-  const field = props.colDef?.field ?? "";
-  const colId = props.colDef?.colId ?? field ?? "";
+  const field = props.colDef?.field ?? '';
+  const colId = props.colDef?.colId ?? field ?? '';
 
   const updateValue = useCallback(
     async (saveFn: (selectedRows: any[]) => Promise<boolean>, tabDirection: 1 | 0 | -1): Promise<boolean> => {

package/src/contexts/GridSubComponentContext.ts

@@ -1,4 +1,4 @@
-import { createContext } from "react";
+import { createContext } from 'react';
 
 export interface GridSubComponentContextType {
   value: any;
@@ -10,16 +10,17 @@ export interface GridSubComponentContextType {
 }
 
 export const GridSubComponentContext = createContext<GridSubComponentContextType>({
-  value: "GridSubComponentContext value no context",
+  value: 'GridSubComponentContext value no context',
   data: {},
   setValue: () => {
-    console.error("GridSubComponentContext setValue no context");
+    console.error('GridSubComponentContext setValue no context');
   },
   setValid: () => {
-    console.error("GridSubComponentContext setValid no context");
+    console.error('GridSubComponentContext setValid no context');
   },
+  // eslint-disable-next-line @typescript-eslint/require-await
   triggerSave: async () => {
-    console.error("GridSubComponentContext triggerSave no context");
+    console.error('GridSubComponentContext triggerSave no context');
   },
   context: null,
 });

package/src/contexts/GridUpdatingContext.tsx

@@ -1,4 +1,4 @@
-import { createContext } from "react";
+import { createContext } from 'react';
 
 export type GridUpdatingContextType = {
   checkUpdating: (fields: string | string[], id: number | string) => boolean;
@@ -13,15 +13,16 @@ export type GridUpdatingContextType = {
 
 export const GridUpdatingContext = createContext<GridUpdatingContextType>({
   checkUpdating: () => {
-    console.error("Missing GridUpdatingContext");
+    console.error('Missing GridUpdatingContext');
     return false;
   },
   updatingCols: () => {
-    console.error("Missing GridUpdatingContext");
+    console.error('Missing GridUpdatingContext');
     return [];
   },
+  // eslint-disable-next-line @typescript-eslint/require-await
   modifyUpdating: async () => {
-    console.error("Missing GridUpdatingContext");
+    console.error('Missing GridUpdatingContext');
   },
   updatedDep: 0,
 });

package/src/contexts/GridUpdatingContextProvider.tsx

@@ -1,7 +1,7 @@
-import { castArray, flatten, remove } from "lodash-es";
-import { PropsWithChildren, useRef, useState } from "react";
+import { castArray, flatten, remove } from 'lodash-es';
+import { PropsWithChildren, useRef, useState } from 'react';
 
-import { GridUpdatingContext } from "./GridUpdatingContext";
+import { GridUpdatingContext } from './GridUpdatingContext';
 
 export type GridUpdatingContextStatus = Record<string, (number | string)[] | undefined>;