fix: HybridSelectionModelOption should all be optional (#1173)

Author: ghiscoding

src/models/selectionModelOption.interface.ts

@@ -15,31 +15,31 @@ export type SelectionType =
 
 export interface HybridSelectionModelOption {
   /** defaults to True, do we want to select the active cell? */
-  selectActiveCell: boolean;
+  selectActiveCell?: boolean;
 
   /** defaults to True, do we want to select the active row? */
-  selectActiveRow: boolean;
+  selectActiveRow?: boolean;
 
   /** cell range selector */
   cellRangeSelector?: SlickCellRangeSelector;
 
   /** Defaults to False, should we select when dragging? */
-  dragToSelect: boolean;
+  dragToSelect?: boolean;
 
   /** Defaults to True, should we auto-scroll when dragging a row */
-  autoScrollWhenDrag: boolean;
+  autoScrollWhenDrag?: boolean;
 
   /** Row Selection on RowMoveManage column */
-  handleRowMoveManagerColumn: boolean;
+  handleRowMoveManagerColumn?: boolean;
 
   /** Row Selection on these columns */
-  rowSelectColumnIds: string[];
+  rowSelectColumnIds?: string[];
 
   /** function to toggle Row Selection Models */
-  rowSelectOverride: RowSelectOverride | undefined;
+  rowSelectOverride?: RowSelectOverride | undefined;
 
   /** Defaults to 'mixed', use a specifc selection type */
-  selectionType: SelectionType;
+  selectionType?: SelectionType;
 }
 
 export interface RowSelectionModelOption {

src/plugins/slick.hybridselectionmodel.ts

@@ -282,7 +282,7 @@ export class SlickHybridSelectionModel implements SelectionModel {
 
     const targetColumn = this._grid.getVisibleColumns()[data.cell];
     if (targetColumn) {
-      return this._options?.rowSelectColumnIds.includes('' + targetColumn.id) || false;
+      return this._options?.rowSelectColumnIds?.includes('' + targetColumn.id) || false;
     }
     return false;
   }