Skip to content

fix(reports): resolve sql group by queries, svg chart geometry and api error handling - #9

Merged
AdityaRaj212 merged 1 commit into
mainfrom
fix/reports-emergency-prod
Oct 1, 2026
Merged

AdityaRaj212 merged 1 commit into
mainfrom
fix/reports-emergency-prod

Conversation

@AdityaRaj212

Copy link
Copy Markdown
Owner

📋 Summary of Changes

Quick fix for the error that we are getting in reports page in prod.

🎯 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.

@AdityaRaj212 AdityaRaj212 self-assigned this Oct 1, 2026
Copilot AI balanced review requested due to automatic review settings October 1, 2026 20:52
@AdityaRaj212 AdityaRaj212 added the bug Something isn't working label Oct 1, 2026
@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 8:52pm UTC

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.

@AdityaRaj212
AdityaRaj212 merged commit 09b4ed3 into main Oct 1, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — edcbf236 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

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants