Skip to content

fix(billing): honor table sort order in per-user lazy load - #430

Open
karpikpl wants to merge 1 commit into
mainfrom
copilot/fix-issue-418
Open

fix(billing): honor table sort order in per-user lazy load#430
karpikpl wants to merge 1 commit into
mainfrom
copilot/fix-issue-418

Conversation

@karpikpl

Copy link
Copy Markdown
Collaborator

Fixes #418

Summary

  • apply v-data-table sort options before selecting lazy-loaded per-user billing rows
  • forward billing sort/page options to the per-user billing endpoint for DB-backed spend ordering
  • add regression coverage for token and spend sort lazy-load requests

Validation

  • npm test -- tests/billing-per-user-lazy-load.spec.ts tests/billing-credit-reader.spec.ts tests/billing-credits.db-first.spec.ts --run
  • npm run build

Note: a full npm test -- --run in the symlinked /tmp worktree failed before running Nuxt-environment suites because @nuxt/test-utils tried to import /@fs/home/.../node_modules/@nuxt/test-utils/dist/runtime/entry.mjs; targeted node suites passed and the production build succeeded.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: bbf27d66-9aca-451b-85ee-3a32e0775d52
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.

Per-user billing lazy-load ignores table sort order — wrong users fetched, visible rows stay $0

1 participant