Skip to content

fix(keep-ui): enable datetime sorting on creation_time column in incidents table (#6857) - #6875

Closed
MyDude92 wants to merge 89 commits into
keephq:mainfrom
MyDude92:fix/issue-6857-incident-table-creation-time-sort
Closed

MyDude92 wants to merge 89 commits into
keephq:mainfrom
MyDude92:fix/issue-6857-incident-table-creation-time-sort

Conversation

@MyDude92

Copy link
Copy Markdown

🎯 Objective

Resolves issue #6857 by enabling client-side datetime sorting on the creation_time ("Created At") column in the incidents table.

Closes #6857

📊 Before vs. After Benchmark

  • Before: Clicking the "Created At" column header in the Incidents Table did not sort the rows because enableSorting: true and sortingFn: "datetime" were omitted from the column definition.
  • After: Users can click the "Created At" column header to toggle ascending and descending order based on timestamp value, using TanStack Table's built-in "datetime" sorting comparator.

🛠️ Precise Diff

  • keep-ui/features/incidents/incident-list/ui/incidents-table.tsx: Added enableSorting: true and sortingFn: "datetime" to the creation_time column definition.

…CLEAN.md $7,500 USD solution and submission dossier
…UARDS.md solutions and dossiers (Bounties 16-19)
…LN_BW.md solutions and dossiers (Bounties 16-19)
…ion.py solutions and dossiers (Bounties 16-19)
…SIER.md solutions and dossiers (Bounties 16-19)
…MASK.md solutions and dossiers (Bounties 20-22)
…LOW.md solutions and dossiers (Bounties 20-22)
@shahargl

Copy link
Copy Markdown
Member

Slop

@shahargl shahargl closed this Oct 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[➕ Feature]: Run a quick sanity check before letting the frontend start

2 participants