Skip to content

fix(reports): resolve unhandled client runtime crashes, sql group by,… - #10

Merged
AdityaRaj212 merged 1 commit into
stagingfrom
fix/reports-crash-staging
Oct 1, 2026
Merged

AdityaRaj212 merged 1 commit into
stagingfrom
fix/reports-crash-staging

Conversation

@AdityaRaj212

Copy link
Copy Markdown
Owner

📋 Summary of Changes

Fix for crashing reports page.

🎯 Target Branch Verification

  • This PR targets the staging branch (Direct PRs to main are reserved exclusively for staging-to-main production releases).

✅ Verification & Quality Checklist

Before requesting review, please confirm:

  • Typecheck: npx tsc --noEmit passes with 0 errors.
  • Test Suite: npm test passes all tests (unit & security tests).
  • Production Build: npm run build succeeds without warnings or compilation errors.
  • Deterministic Financial Math: All currency math is conducted in integer minor units (paise/cents). No floating-point rounding drift.
  • Security & Authorization: All newly added or modified API endpoints strictly validate sessions and prevent cross-tenant IDOR.
  • Design Aesthetics & Theming: Tested in both Light and Dark mode, responsive on desktop and mobile viewports.

📸 Screenshots or Recordings (if applicable)

Add before/after screenshots or screen recordings demonstrating the visual changes.

Copilot AI balanced review requested due to automatic review settings October 1, 2026 21:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vercel

vercel Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
bagstack Ready Ready Preview Oct 1, 2026 9:22pm UTC

@AdityaRaj212
AdityaRaj212 merged commit 2bf91f4 into staging Oct 1, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 71f58d55 Deployed Oct 1, 2026 by vercel[bot]
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.

2 participants