@adaptabletools/adaptable-cjs 21.0.6 → 21.0.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adaptabletools/adaptable-cjs",
-  "version": "21.0.6",
+  "version": "21.0.8",
   "description": "Powerful data-agnostic HTML5 AG Grid extension which provides advanced, cutting-edge functionality to meet all DataGrid requirements",
   "keywords": [
     "web-components",
@@ -54,6 +54,7 @@
     "react-select": "5.10.1",
     "react-toastify": "9.1.3",
     "rebass": "^3.2.2",
+    "@types/rebass": "^4.0.10",
     "redux": "^5.0.1",
     "rxjs": "^7.4.0",
     "sentence-case": "^3.0.4",

package/src/AdaptableOptions/DateInputOptions.d.ts

@@ -11,7 +11,7 @@ export interface DateInputOptions {
     /**
      * Format string for formatting date input field
      *
-     * @defaultValue 'yyyy-MM-dd'
+     * @defaultValue 'yyyy-MM-dd' (ISO 8601 format)
      */
     dateFormat?: string;
     /**

package/src/Api/Internal/AdaptableInternalApi.d.ts

@@ -123,4 +123,5 @@ export declare class AdaptableInternalApi extends ApiBase {
     findAdaptableObjectsByLookupCriteria<T extends AdaptableObjectWithScope>({ scope, tag, ids }: AdaptableObjectLookupCriteria, specificAdaptableObjects: T[]): T[];
     buildBaseContext(): BaseContext;
     setCellSummaryInfo(cellSummaryInfo: CellSummmaryInfo): void;
+    parseDateValue(dateValue: string | Date | number): Date | undefined;
 }

package/src/Api/Internal/AdaptableInternalApi.js

@@ -9,6 +9,7 @@ const StringExtensions_1 = tslib_1.__importDefault(require("../../Utilities/Exte
 const GeneralConstants_1 = require("../../Utilities/Constants/GeneralConstants");
 const waitForCondition_1 = require("../../Utilities/waitForCondition");
 const Icon_1 = require("../../components/Icon");
+const DateHelper_1 = require("../../Utilities/Helpers/DateHelper");
 class AdaptableInternalApi extends ApiBase_1.ApiBase {
     getInternalState() {
         return this.getAdaptableState().Internal;
@@ -443,5 +444,13 @@ class AdaptableInternalApi extends ApiBase_1.ApiBase {
     setCellSummaryInfo(cellSummaryInfo) {
         this.dispatchAction(InternalRedux.SetCellSummaryInfo(cellSummaryInfo));
     }
+    parseDateValue(dateValue) {
+        const dateFormat = this.getUserInterfaceOptions().dateInputOptions.dateFormat;
+        if (dateFormat === 'yyyy-MM-dd') {
+            // if the format is ISO, we don't pass it so that we use the date-fns `parseISO` which is more robust
+            return (0, DateHelper_1.parseDateValue)(dateValue);
+        }
+        return (0, DateHelper_1.parseDateValue)(dateValue, dateFormat);
+    }
 }
 exports.AdaptableInternalApi = AdaptableInternalApi;

package/src/Api/Internal/ColumnInternalApi.d.ts

@@ -1,7 +1,5 @@
 import { ApiBase } from '../Implementation/ApiBase';
-import { ColDef, Column, HeaderValueGetterParams, IRowNode } from 'ag-grid-enterprise';
-import { CustomSort } from '../../AdaptableState/CustomSortState';
-import { ColumnValuesComparer } from '../../AdaptableOptions/CustomSortOptions';
+import { ColDef, Column, HeaderValueGetterParams } from 'ag-grid-enterprise';
 import { AdaptableColumn, AdaptableColumnDataType } from '../../types';
 export declare function getAutoRowGroupColumnIdFor(columnId: string): string;
 export declare class ColumnInternalApi extends ApiBase {
@@ -29,7 +27,6 @@ export declare class ColumnInternalApi extends ApiBase {
      * @param columnId columnId to look up
      */
     getAgGridColumnForAdaptableColumn(columnId: string): Column;
-    getActiveColumnComparator(columnId: string, customSort?: CustomSort, customSortComparer?: ColumnValuesComparer): (valueA: any, valueB: any, nodeA?: IRowNode, nodeB?: IRowNode, isInverted?: boolean) => number | undefined;
     isSpecialColumn(columnId: string, column?: AdaptableColumn): boolean;
     getColumnHeaderName(params: HeaderValueGetterParams): string;
     private buildColumnHeaderContext;

package/src/Api/Internal/ColumnInternalApi.js

@@ -69,18 +69,6 @@ class ColumnInternalApi extends ApiBase_1.ApiBase {
     getAgGridColumnForAdaptableColumn(columnId) {
         return this._adaptable.getAgGridColumnForColumnId(columnId);
     }
-    getActiveColumnComparator(columnId, customSort, customSortComparer) {
-        if ((!customSort || customSort?.IsSuspended) && !customSortComparer) {
-            // defaults to AG-Grid column definition comparator if no CustomSort is defined&active
-            const colDefComparator = this._adaptable.agGridColumnAdapter.getUserColDefProperty(columnId, 'comparator');
-            return colDefComparator;
-        }
-        // CustomSort Comparer function takes precedence over CustomSort SortedValues
-        const comparerFunction = customSortComparer
-            ? customSortComparer.comparer
-            : this.getCustomSortApi().internalApi.getDefaultCustomSortComparer(customSort.ColumnId, customSort.SortedValues);
-        return comparerFunction;
-    }
     isSpecialColumn(columnId, column = null) {
         if (column) {
             return column.isCalculatedColumn || column.isFreeTextColumn || column.isActionColumn;

package/src/Api/Internal/ExportInternalApi.d.ts

@@ -14,6 +14,7 @@ export declare class ExportInternalApi extends ApiBase {
     setExportInProgress(reportName: ReportNameType, reportFormat: ReportFormatType, exportDestination: ExportDestinationType): void;
     setExportComplete(): void;
     isVisualDataExportInProgress(): boolean;
+    isExcelExportInProgress(): boolean;
     createSystemReport(systemReportName: SystemReportName): Report;
     isSystemReport(reportName: ReportNameType): boolean;
     isSystemDestination(destination: ExportDestinationType): boolean;

package/src/Api/Internal/ExportInternalApi.js

@@ -86,6 +86,9 @@ class ExportInternalApi extends ApiBase_1.ApiBase {
     isVisualDataExportInProgress() {
         return this.getAdaptableState().Internal.Export.inProgress?.reportFormat === 'VisualExcel';
     }
+    isExcelExportInProgress() {
+        return this.getAdaptableState().Internal.Export.inProgress?.reportFormat === 'Excel';
+    }
     createSystemReport(systemReportName) {
         switch (systemReportName) {
             case GeneralConstants_1.ALL_DATA_REPORT:

package/src/Redux/Store/AdaptableStore.d.ts

@@ -5,8 +5,6 @@ import { BaseState } from '../../AdaptableState/BaseState';
 import { IAdaptableStore, LoadStoreConfig } from './Interface/IAdaptableStore';
 type EmitterCallback = (data?: any) => any;
 type EmitterAnyCallback = (eventName: string, data?: any) => any;
-export declare const INIT_STATE = "INIT_STATE";
-export declare const LOAD_STATE = "LOAD_STATE";
 export interface ResetUserDataAction extends Redux.Action {
 }
 export interface InitStateAction extends Redux.Action {

package/src/Redux/Store/AdaptableStore.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.AdaptableStore = exports.LoadState = exports.InitState = exports.LOAD_STATE = exports.INIT_STATE = void 0;
+exports.AdaptableStore = exports.LoadState = exports.InitState = void 0;
 const tslib_1 = require("tslib");
 const Redux = tslib_1.__importStar(require("redux"));
 const PluginsRedux = tslib_1.__importStar(require("../ActionsReducers/PluginsRedux"));
@@ -44,13 +44,13 @@ const ThemeRedux = tslib_1.__importStar(require("../ActionsReducers/ThemeRedux")
 const ToolPanelRedux = tslib_1.__importStar(require("../ActionsReducers/ToolPanelRedux"));
 const TeamSharingState_1 = require("../../AdaptableState/TeamSharingState");
 const buildAdaptableStateFunctionConfig_1 = require("./buildAdaptableStateFunctionConfig");
-exports.INIT_STATE = 'INIT_STATE';
-exports.LOAD_STATE = 'LOAD_STATE';
+const INIT_STATE = 'INIT_STATE';
+const LOAD_STATE = 'LOAD_STATE';
 const NON_PERSIST_ACTIONS = {
     '@@INIT': true,
     '@@redux/init': true,
-    [exports.LOAD_STATE]: true,
-    [exports.INIT_STATE]: true,
+    [LOAD_STATE]: true,
+    [INIT_STATE]: true,
     // progress indicators should NOT interfere with state management as it may lead to race conditions due to load/persist state being async
     [PopupRedux_1.PROGRESS_INDICATOR_SHOW]: true,
     [PopupRedux_1.PROGRESS_INDICATOR_HIDE]: true,
@@ -62,11 +62,11 @@ const NON_PERSISTENT_STORE_KEYS = [
     'Plugins',
 ];
 const InitState = () => ({
-    type: exports.INIT_STATE,
+    type: INIT_STATE,
 });
 exports.InitState = InitState;
 const LoadState = (State) => ({
-    type: exports.LOAD_STATE,
+    type: LOAD_STATE,
     State,
 });
 exports.LoadState = LoadState;
@@ -167,7 +167,7 @@ class AdaptableStore {
         Redux.combineReducers(rootReducerObject);
         const rootReducerWithResetManagement = (state, action) => {
             switch (action.type) {
-                case exports.LOAD_STATE:
+                case LOAD_STATE:
                     const { State } = action;
                     Object.keys(State).forEach((key) => {
                         state[key] = State[key];
@@ -199,7 +199,7 @@ class AdaptableStore {
             });
         };
         // this is now VERY BADLY NAMED!
-        let rootReducer = (0, AdaptableReduxMerger_1.mergeReducer)(rootReducerWithResetManagement, exports.LOAD_STATE);
+        let rootReducer = (0, AdaptableReduxMerger_1.mergeReducer)(rootReducerWithResetManagement, LOAD_STATE);
         const composeEnhancers = (x) => x;
         const persistedReducer = (state, action) => {
             if (adaptable.isDestroyed) {
@@ -1507,7 +1507,7 @@ const adaptableMiddleware = (adaptable) => (function(middlewareAPI) {
                 /*******************
                  * MANAGING STATE ACTIONS
                  *******************/
-                case exports.INIT_STATE: {
+                case INIT_STATE: {
                     let returnAction = next(action);
                     if (adaptable.isReady) {
                         // we need this here for when the state key is changed

package/src/Utilities/Helpers/DateHelper.d.ts

@@ -8,4 +8,3 @@ export declare const isValueValidDate: (data: any) => boolean;
 export declare const dateToISO: (date: Date | number | string) => string;
 export declare const parseToISO: (date: string | Date | number, dateFormat?: string) => string;
 export declare const parseDateValue: (dateValue: string | Date | number, dateFormat?: string) => Date | undefined;
-export declare const parseFilterInputDate: (stringDate: string) => Date;

package/src/Utilities/Helpers/DateHelper.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.parseFilterInputDate = exports.parseDateValue = exports.parseToISO = exports.dateToISO = exports.isValueValidDate = exports.isValidDate = void 0;
+exports.parseDateValue = exports.parseToISO = exports.dateToISO = exports.isValueValidDate = exports.isValidDate = void 0;
 const date_fns_1 = require("date-fns");
 const FormatHelper_1 = require("./FormatHelper");
 const AdaptableLogger_1 = require("../../agGrid/AdaptableLogger");
@@ -30,6 +30,9 @@ const parseToISO = (date, dateFormat) => {
     return (0, exports.isValidDate)(dateInstance) ? (0, exports.dateToISO)(dateInstance) : '';
 };
 exports.parseToISO = parseToISO;
+// !!!
+// AFL: ideally we should NEVER call directly this method, but use the AdaptableInternalApi.parseDateValue() instead
+// we should refactor this with the first opportunity
 const parseDateValue = (dateValue, dateFormat) => {
     if (dateValue == undefined || (typeof dateValue === 'string' && dateValue.trim() === '')) {
         return undefined;
@@ -37,29 +40,37 @@ const parseDateValue = (dateValue, dateFormat) => {
     if (dateValue instanceof Date) {
         return !isNaN(dateValue.getTime()) ? dateValue : undefined;
     }
-    let dateInstance;
-    if (typeof dateValue === 'number') {
-        dateInstance = new Date(dateValue);
-    }
-    else {
-        // typeof dateValue === 'string'
-        if (dateFormat) {
-            dateInstance = (0, date_fns_1.parse)(dateValue, dateFormat, new Date());
+    try {
+        let dateInstance;
+        if (typeof dateValue === 'number') {
+            dateInstance = new Date(dateValue);
         }
         else {
-            dateInstance = (0, date_fns_1.parseISO)(dateValue);
+            // typeof dateValue === 'string'
+            if (dateFormat) {
+                dateInstance = (0, date_fns_1.parse)(dateValue, dateFormat, new Date());
+            }
+            else {
+                dateInstance = (0, date_fns_1.parseISO)(dateValue);
+            }
+            if (!(0, exports.isValidDate)(dateInstance)) {
+                // last chance: try to use the native Date.parse(), https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/parse
+                dateInstance = new Date(Date.parse(dateValue));
+            }
         }
         if (!(0, exports.isValidDate)(dateInstance)) {
-            // last chance: try to use the native Date.parse(), https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/parse
-            dateInstance = new Date(Date.parse(dateValue));
+            AdaptableLogger_1.AdaptableLogger.consoleWarnBase(`Invalid date value "${dateValue}" - cannot be converted to a Date instance. Please make sure you specify adaptableOptions.userInterfaceOptions.dateInputOptions.dateFormat correctly.`);
+            dateInstance = undefined;
         }
+        return dateInstance;
     }
-    if (!(0, exports.isValidDate)(dateInstance)) {
-        AdaptableLogger_1.AdaptableLogger.consoleWarnBase(`Invalid date value "${dateValue}" - cannot be converted to a Date instance. Please make sure you specify adaptableOptions.userInterfaceOptions.dateInputOptions.dateFormat correctly.`);
-        dateInstance = undefined;
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        const errorStack = error instanceof Error ? error.stack : undefined;
+        AdaptableLogger_1.AdaptableLogger.consoleErrorBase(`Error parsing date value "${dateValue}": ${errorMessage}`, {
+            stack: errorStack,
+            originalError: error,
+        });
     }
-    return dateInstance;
 };
 exports.parseDateValue = parseDateValue;
-const parseFilterInputDate = (stringDate) => new Date(stringDate);
-exports.parseFilterInputDate = parseFilterInputDate;

package/src/View/Filter/FilterViewPanel.js

@@ -35,7 +35,7 @@ class FilterViewPanelComponent extends React.Component {
                 ArrayExtensions_1.ArrayExtensions.IsNotNullOrEmpty(this.props.ColumnFilters) && (React.createElement(AdaptablePopover_1.AdaptablePopover, { popupPadding: 0, className: `ab-${elementType}__Filter__info`, headerText: "", bodyText: [React.createElement(ActiveFiltersPanel_1.ActiveFiltersPanel, null)], useButton: true, showEvent: 'focus', hideEvent: "blur", popoverMinWidth: 400 })),
                 React.createElement(ButtonClear_1.ButtonClear, { "aria-label": 'Clear Filters', className: `ab-${elementType}__Filter__clear`, marginLeft: 1, marginBottom: 0, marginRight: 1, onClick: () => this.onClearFilters(), tooltip: "Clear Filters", disabled: this.props.ColumnFilters.length == 0, showText: this.props.viewType === 'ToolPanel' }, this.props.viewType === 'ToolPanel' && 'Clear'),
                 React.createElement(SimpleButton_1.default, { "aria-label": isAtLeastOneFilterActive ? 'Suspend All Filters' : 'Resume All Filters', className: (0, join_1.default)(`ab-${elementType}__Filter__suspend-button`, isAtLeastOneFilterActive && `ab-${elementType}__Filter__suspend-all`, !isAtLeastOneFilterActive && `ab-${elementType}__Filter__un-suspend-all`), disabled: !isAtLeastOneFilter, onClick: handleSuspendUnsuspendAll, tone: isAtLeastOneFilterActive ? 'neutral' : 'success', variant: "text", icon: isAtLeastOneFilterActive ? 'pause' : 'play', accessLevel: this.props.accessLevel })),
-            React.createElement(rebass_1.Flex, { alignItems: "center" }, this.props.api.filterApi.columnFilterApi.isQuickFilterAvailable() && (React.createElement(CheckBox_1.CheckBox, { className: `ab-${elementType}__Filter__active-check`, disabled: this.props.accessLevel === 'ReadOnly' ||
+            React.createElement(rebass_1.Flex, { alignItems: "center" }, this.props.api.filterApi.columnFilterApi.isQuickFilterAvailable() && (React.createElement(CheckBox_1.CheckBox, { "data-name": "quick-filter-toggle", className: `ab-${elementType}__Filter__active-check`, disabled: this.props.accessLevel === 'ReadOnly' ||
                     this.props.api.layoutApi.isCurrentLayoutPivot(), marginTop: 0, marginBottom: 0, fontSize: 2, padding: 1, checked: this.props.IsQuickFilterVisible, onChange: (checked) => {
                     checked ? this.props.onShowQuickFilterBar() : this.props.onHideQuickFilterBar();
                 } }, "Filter Bar")))));

package/src/agGrid/AdaptableAgGrid.js

@@ -1089,6 +1089,24 @@ You need to define at least one Layout!`);
                     color: '#aaaaaa',
                     pattern: 'Solid',
                 },
+            }, 
+            // see #EXCEL_EXPORT_DATA_TYPES
+            {
+                id: 'stringExcelType',
+                dataType: 'String',
+            }, {
+                id: 'booleanExcelType',
+                dataType: 'Boolean',
+            }, {
+                id: 'dateExcelType',
+                dataType: 'DateTime',
+            }, {
+                id: 'numberExcelType',
+                // dataType: 'Number',
+                // AG Grid requires either dataType or numberFormat to be set for Numbers
+                numberFormat: {
+                    format: '0.###############',
+                },
             });
             this.agGridExportAdapter.originalExcelStyles = excelStyles;
             this.agGridExportAdapter.DANGER_excelStyles = this.agGridExportAdapter.originalExcelStyles;
@@ -3199,11 +3217,6 @@ You need to define at least one Layout!`);
         return this.agGridModulesAdapter.isAgGridModuleRegistered('CsvExportModule');
     }
     isQuickFilterAvailable() {
-        if (this.api.layoutApi.isCurrentLayoutPivot() &&
-            this.adaptableOptions.filterOptions.useAdaptableFiltering) {
-            // hide completely the quick filter if pivot is enabled
-            return false;
-        }
         return this.hasFloatingFilterOnAtLeastOneColumn(this.agGridAdapter.getAgGridApi().getColumnDefs());
     }
     hasFloatingFilterOnAtLeastOneColumn(columnDefs) {

package/src/agGrid/AgGridColumnAdapter.js

@@ -174,11 +174,27 @@ class AgGridColumnAdapter {
                 if (!gridCell.column) {
                     return null;
                 }
-                // if a VisualExcel report format export is in progress, we are interested only in the Excel Style Class
-                if (this.adaptableApi.exportApi.internalApi.isVisualDataExportInProgress()) {
+                const isExcelExport = this.adaptableApi.exportApi.internalApi.isExcelExportInProgress();
+                const isVisualDataExport = this.adaptableApi.exportApi.internalApi.isVisualDataExportInProgress();
+                if (isExcelExport || isVisualDataExport) {
+                    const excelStyleClasses = [];
                     const userDefinedCellClass = typeof userCellClass === 'function' ? userCellClass(params) : userCellClass;
-                    const cellClassKey = AgGridExportAdapter_1.AgGridExportAdapter.getExcelClassNameForCell(colId, gridCell.primaryKeyValue, userDefinedCellClass);
-                    return this.adaptableInstance.agGridExportAdapter.getExcelStyleIdForCellClassKey(cellClassKey);
+                    if (userDefinedCellClass) {
+                        if (Array.isArray(userDefinedCellClass)) {
+                            excelStyleClasses.push(...userDefinedCellClass.filter(Boolean));
+                        }
+                        else {
+                            excelStyleClasses.push(userDefinedCellClass);
+                        }
+                    }
+                    if (isVisualDataExport) {
+                        const cellClassKey = AgGridExportAdapter_1.AgGridExportAdapter.getExcelClassNameForCell(colId, gridCell.primaryKeyValue, userDefinedCellClass);
+                        const customCellClass = this.adaptableInstance.agGridExportAdapter.getExcelStyleIdForCellClassKey(cellClassKey);
+                        if (customCellClass) {
+                            excelStyleClasses.push(customCellClass);
+                        }
+                    }
+                    return excelStyleClasses.length ? excelStyleClasses : null;
                 }
                 const isQuickSearchActive = this.isQuickSearchActive(gridCell);
                 const editableClassName = this.getEditableCellClass(gridCell, params);
@@ -617,7 +633,7 @@ class AgGridColumnAdapter {
         const isFloatingFilterDisabled = !colDef.filter ||
             !colDef.floatingFilter ||
             !this.adaptableOptions.filterOptions.useAdaptableFiltering ||
-            !this.adaptableOptions.filterOptions.columnFilterOptions.showQuickFilter;
+            !this.adaptableApi.filterApi.columnFilterApi.isQuickFilterVisible();
         if (this.adaptableApi.columnApi.isAutoRowGroupColumn(col.getColId())) {
             this.setColDefProperty(col, 'floatingFilter', (original_floatingFilter) => {
                 // the floating filter for the group column is "inherited" from the base column
@@ -808,10 +824,47 @@ class AgGridColumnAdapter {
     }
     setupColumnComparator({ col, colId, abColumn }) {
         const customSort = this.adaptableApi.customSortApi.getCustomSortForColumn(colId);
-        const columnSortComparer = this.adaptableApi.customSortApi.internalApi.getCustomSortComparer(abColumn.columnId);
+        const customSortComparer = this.adaptableApi.customSortApi.internalApi.getCustomSortComparer(abColumn.columnId);
+        const memoizedDateValues = new Map();
+        const getDateValue = (rawDateValue) => {
+            if (memoizedDateValues.has(rawDateValue)) {
+                return memoizedDateValues.get(rawDateValue);
+            }
+            const dateValue = this.adaptableApi.internalApi.parseDateValue(rawDateValue);
+            memoizedDateValues.set(rawDateValue, dateValue);
+            return dateValue;
+        };
         const comparatorGetter = (propName) => {
-            return () => {
-                return this.adaptableApi.columnApi.internalApi.getActiveColumnComparator(colId, customSort, columnSortComparer);
+            return (userPropertyValue) => {
+                // CustomSort Comparer function takes precedence over CustomSort SortedValues
+                if (customSortComparer) {
+                    return customSortComparer.comparer;
+                }
+                if (customSort && !customSort.IsSuspended) {
+                    return this.adaptableApi.customSortApi.internalApi.getDefaultCustomSortComparer(customSort.ColumnId, customSort.SortedValues);
+                }
+                if (userPropertyValue) {
+                    return userPropertyValue;
+                }
+                // for DATE columns we have to use the normalised values (Date objects) to compare
+                if (abColumn.dataType === 'date') {
+                    return (valueA, valueB) => {
+                        const dateA = getDateValue(valueA);
+                        const dateB = getDateValue(valueB);
+                        if (dateA && dateB) {
+                            return dateA.getTime() - dateB.getTime();
+                        }
+                        else if (dateA) {
+                            return 1; // consider non-null dates as greater than null/undefined
+                        }
+                        else if (dateB) {
+                            return -1; // consider non-null dates as greater than null/undefined
+                        }
+                        else {
+                            return 0; // both are null/undefined, considered equal
+                        }
+                    };
+                }
             };
         };
         this.setColDefProperty(col, 'comparator', comparatorGetter('comparator'));

package/src/agGrid/AgGridExportAdapter.js

@@ -606,6 +606,27 @@ class AgGridExportAdapter {
                 this.registerExcelStyle(finalCellExcelStyle, cellClassId);
             });
         }, forAllVisibleRowNodesDoConfig);
+        // see #EXCEL_EXPORT_DATA_TYPES
+        this.excelStylesCache['stringExcelType'] = {
+            id: 'stringExcelType',
+            dataType: 'String',
+        };
+        this.excelStylesCache['booleanExcelType'] = {
+            id: 'booleanExcelType',
+            dataType: 'Boolean',
+        };
+        this.excelStylesCache['dateExcelType'] = {
+            id: 'dateExcelType',
+            dataType: 'DateTime',
+        };
+        this.excelStylesCache['numberExcelType'] = {
+            id: 'numberExcelType',
+            // dataType: 'Number',
+            // AG Grid requires either dataType or numberFormat to be set for Numbers
+            numberFormat: {
+                format: '0.###############',
+            },
+        };
         return Object.values(this.excelStylesCache);
     }
     registerExcelStyle(excelStyle, cellClassKey) {

package/src/env.js

@@ -2,6 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = {
     NEXT_PUBLIC_INFINITE_TABLE_LICENSE_KEY: "StartDate=2021-06-29|EndDate=2030-01-01|Owner=Adaptable|Type=distribution|TS=1624971462479|C=137829811,1004007071,2756196225,1839832928,3994409405,636616862" || '',
-    PUBLISH_TIMESTAMP: 1759506326299 || Date.now(),
-    VERSION: "21.0.6" || '--current-version--',
+    PUBLISH_TIMESTAMP: 1761108119347 || Date.now(),
+    VERSION: "21.0.8" || '--current-version--',
 };

package/src/layout-manager/src/index.js

@@ -336,6 +336,8 @@ class LayoutManager extends LMEmitter_1.LMEmitter {
     getPivotLayoutModelFromGrid() {
         const pivotResultColumns = this.gridApi.getPivotResultColumns() || [];
         const pivotResultColumnsSet = new Set(pivotResultColumns.map((col) => col.getColId()));
+        const rowGroupColumns = this.gridApi.getRowGroupColumns() || [];
+        const rowGroupColumnsSet = new Set(rowGroupColumns.map((col) => col.getColId()));
         const prevLayout = this.currentLayout;
         const columnState = this.gridApi
             .getColumnState()
@@ -343,6 +345,7 @@ class LayoutManager extends LMEmitter_1.LMEmitter {
         let PivotColumns = this.gridApi.getPivotColumns().map((col) => col.getColId());
         const layout = this.getUndecidedLayoutModelFromGrid(columnState);
         let ColumnSizing = layout.ColumnSizing || {};
+        let ColumnVisibility = { ...layout.ColumnVisibility };
         //let's also include the column widths of the pivotResult columns
         pivotResultColumns.forEach((col) => {
             const colId = col.getColId();
@@ -358,6 +361,18 @@ class LayoutManager extends LMEmitter_1.LMEmitter {
         if (!Object.keys(ColumnSizing).length) {
             ColumnSizing = undefined;
         }
+        // from column visibility, let's remove all columns that are not actually in the pivot layout
+        // since having normal table columns here will break the equality check
+        // so we only want to keep the group columns and the pivot result columns
+        Object.keys(ColumnVisibility).forEach((colId) => {
+            const isInLayout = pivotResultColumnsSet.has(colId) || rowGroupColumnsSet.has(colId);
+            if (!isInLayout) {
+                delete ColumnVisibility[colId];
+            }
+        });
+        if (!Object.keys(ColumnVisibility).length) {
+            ColumnVisibility = undefined;
+        }
         delete layout.TableColumns;
         const pivotLayout = {
             Name: layout.Name,
@@ -375,7 +390,7 @@ class LayoutManager extends LMEmitter_1.LMEmitter {
             ColumnPinning: layout.ColumnPinning,
             ColumnSorts: layout.ColumnSorts,
             ColumnSizing,
-            ColumnVisibility: layout.ColumnVisibility,
+            ColumnVisibility,
             RowGroupValues: layout.RowGroupValues,
             ColumnGroupValues: layout.ColumnGroupValues,
             PivotGroupedColumns: layout.RowGroupedColumns,

package/src/layout-manager/src/normalizeLayoutModel.js

@@ -105,6 +105,11 @@ function normalizeTableLayoutModel(layout, options) {
             };
         }
     }
+    // there might be a case where there are we have an RowGroupedColumns array, but it's empty (or it's inexistent altogether)
+    // and there is no display type specified - so default it to single
+    if (!layout.RowGroupDisplayType && layout.TableColumns) {
+        layout.RowGroupDisplayType = 'single';
+    }
     if (options?.isTree && !layout.TableColumns.includes(exports.AUTO_GROUP_COLUMN_ID__SINGLE)) {
         layout.TableColumns.unshift(exports.AUTO_GROUP_COLUMN_ID__SINGLE);
     }
@@ -161,9 +166,9 @@ function normalizePivotLayoutModel(layout) {
         // make it an own property
         layout.PivotColumnTotal = undefined;
     }
-    if (layout.PivotGroupedColumns && layout.PivotGroupedColumns.length) {
-        layout.RowGroupDisplayType = layout.RowGroupDisplayType || 'single';
-    }
+    // if (layout.PivotGroupedColumns && layout.PivotGroupedColumns.length) {
+    layout.RowGroupDisplayType = layout.RowGroupDisplayType || 'single';
+    // }
     return layout;
 }
 exports.normalizePivotLayoutModel = normalizePivotLayoutModel;