Skip to content
This repository was archived by the owner on Jul 31, 2026. It is now read-only.

fix(table): sort enrolled employees first - #355

Closed
svadrutk wants to merge 1 commit into
mainfrom
cursor/sort-enrolled-employees-first-cc5e
Closed

fix(table): sort enrolled employees first#355
svadrutk wants to merge 1 commit into
mainfrom
cursor/sort-enrolled-employees-first-cc5e

Conversation

@svadrutk

@svadrutk svadrutk commented Jul 20, 2026

Copy link
Copy Markdown

Summary

  • order true boolean sort values before false in ascending table sorts
  • preserve the inverse ordering for descending sorts
  • add regression coverage for enrolled versus unenrolled employees

Test plan

  • pnpm test -- --run src/components/Table/sorting.test.ts (68 tests passed)
  • pnpm type-check
  • pnpm lint

Linear Issue: S-503

Open in WebOpen in Cursor

Co-authored-by: Swad K. <svadrutk@users.noreply.github.com>
@linear-code

linear-code Bot commented Jul 20, 2026

Copy link
Copy Markdown

S-503

@svadrutk
svadrutk marked this pull request as ready for review July 20, 2026 20:41

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@svadrutk svadrutk closed this Jul 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants