Skip to content

Commit

Permalink
fix: arrIncludes autoremove filterFn (#5623)
Browse files Browse the repository at this point in the history
The `filterValue` for this function is not an array. Therefore, we should not check for its length.
  • Loading branch information
lukebui authored Feb 11, 2025
1 parent b1506a7 commit 2efaf57
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packages/table-core/src/filterFns.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ const arrIncludes: FilterFn<any> = (
return row.getValue<unknown[]>(columnId)?.includes(filterValue)
}

arrIncludes.autoRemove = (val: any) => testFalsey(val) || !val?.length
arrIncludes.autoRemove = (val: any) => testFalsey(val)

const arrIncludesAll: FilterFn<any> = (
row,
Expand Down

0 comments on commit 2efaf57

Please # to comment.