table icon indicating copy to clipboard operation
table copied to clipboard

chore: Remove unnecessary `console.error` log from `table.getColumn(id)` implementation

Open mikekobit-stripe opened this issue 10 months ago • 1 comments

The getColumn method is already typed to return Column<TData, unknown> | undefined, indicating to callers that the result might be undefined. The current console.error logging when a column is not found:

  1. Adds noise in development environments
  2. Can mask actual issues
  3. Forces users to implement workarounds (as discussed in https://github.com/TanStack/table/discussions/5505)

This change removes the error logging while maintaining the expected return type, allowing consumers to handle undefined columns as already expected by the API.

mikekobit-stripe avatar Mar 13 '25 14:03 mikekobit-stripe

Is anybody available to review this?

mikekobit-stripe avatar May 29 '25 17:05 mikekobit-stripe